ntg-context - mailing list for ConTeXt users
 help / color / mirror / Atom feed
From: Philipp Gesang <phg@phi-gamma.net>
To: mailing list for ConTeXt users <ntg-context@ntg.nl>
Subject: Re: [font-otn] bug in node mode
Date: Tue, 25 Mar 2014 07:59:13 +0100	[thread overview]
Message-ID: <20140325065913.GA2956@acheron> (raw)
In-Reply-To: <5330D907.5090205@wxs.nl>


[-- Attachment #1.1: Type: text/plain, Size: 1324 bytes --]

···<date: 2014-03-25, Tuesday>···<from: Hans Hagen>···

> On 3/24/2014 8:40 PM, Philipp Gesang wrote:
> > Hi Hans,
> >
> > I’m afraid I have to bother you again with this issue. Consider
> > this code:
> >
> >      \font \mainface = "file:EBGaramond12-Regular.otf:mode=node"
> >      \mainface foo
> >      \bye
> >
> > This breaks with the latest fontloader because in font-otn
> >
> >      local function initialize(sequence,script,language,enabled)
> >          local features = sequence.features
> >          if features then
> >              local order = features.order
> >              for i=1,#order do --
> >              ...
> >
> > the table “features” never has a field “order”; “sequence” has
> > one occasionally, though not always.
> 
> should be
> 
> local function initialize(sequence,script,language,enabled)
>      local features = sequence.features
>      if features then
>          local order = sequence.order
>          if order then
>              for i=1,#order do
>                  ....
>              end
>          else
>              -- can't happen
>          end
>      end
>      return false
> end
> 
> went unnoticed as context uses a variant

This time it’s exactly as I patched it =)

Thanks for fixing,
Philipp


[-- Attachment #1.2: Type: application/pgp-signature, Size: 490 bytes --]

[-- Attachment #2: Type: text/plain, Size: 485 bytes --]

___________________________________________________________________________________
If your question is of interest to others as well, please add an entry to the Wiki!

maillist : ntg-context@ntg.nl / http://www.ntg.nl/mailman/listinfo/ntg-context
webpage  : http://www.pragma-ade.nl / http://tex.aanhet.net
archive  : http://foundry.supelec.fr/projects/contextrev/
wiki     : http://contextgarden.net
___________________________________________________________________________________

      reply	other threads:[~2014-03-25  6:59 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-14  6:48 Philipp Gesang
2014-03-14 11:02 ` Hans Hagen
2014-03-14 17:19   ` Philipp Gesang
2014-03-24 19:40   ` Philipp Gesang
2014-03-25  1:16     ` Hans Hagen
2014-03-25  6:59       ` Philipp Gesang [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140325065913.GA2956@acheron \
    --to=phg@phi-gamma.net \
    --cc=ntg-context@ntg.nl \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).