ntg-context - mailing list for ConTeXt users
 help / color / mirror / Atom feed
From: Hans Hagen <pragma@wxs.nl>
To: mailing list for ConTeXt users <ntg-context@ntg.nl>
Subject: Re: [luatex-plain] disappearing math-on node
Date: Sun, 06 Apr 2014 00:44:41 +0200	[thread overview]
Message-ID: <53408759.9060302@wxs.nl> (raw)
In-Reply-To: <20140405131233.GA16830@acheron>

On 4/5/2014 3:12 PM, Philipp Gesang wrote:
> ···<date: 2014-04-03, Thursday>···<from: Hans Hagen>···
>
>> On 4/3/2014 6:29 PM, Dohyun Kim wrote:
>>> This is a bug report based on the issue discussed at
>>> https://github.com/lualatex/luaotfload/issues/212
>>>
>>> \font\tenrm{file:lmroman10-regular.otf:mode=node;script=latn}\tenrm
>>> \setbox0\vbox{%
>>>     x\penalty-10000
>>>     $a$x$a$
>>> }
>>> \unvbox0
>>> \setbox0\lastbox
>>> \unhbox0
>>> \end
>>>
>>> The plain tex code shown above fails with a lua error:
>>> luatex-fonts-merged.lua:9616: attempt to index local 'current' (a nil value)
>>>
>>> My guess is: as math-on (math-off too) is a "discardable" item after a
>>> linebreak, the first math-on node has gone away. So the
>>> math-on/math-off pair has become broken, by which the behavior of
>>> "node.end_of_math" is confused.
>>
>> mtxrun --script plain test.tex
>>
>> runs ok here so i need an example that fails with context or luatex-plain
>
> It fails with the luatex-plain format from the minimals as of
> yesterday and Luatex 4972:
>
>      error: .../context/tex/texmf-context/tex/context/base/font-ota.lua:144: attempt to index local 'current' (a nil value)
>      .
>      \bye ^^@-\par
>                  \vfill \supereject \end
>      l.17 \bye
>
> Though the issue arises only in node mode.
>
>> normally the font callback is called on a hlist and there are always
>> paired math nodes
>>
>> (what is line 9616 in the current beta?)
>
> Line 9626 with yesterday’s code. Attached are patches for
> font-ota.lua and luatex-fonts-otn.lua that guard all invocations
> of end_of_math() with checks for the subtype.

I think that the removed math node is a bad thing so that will change in 
a next luatex release (i tested a patch but it will not go in the tl 
release). (Just like the math node at the end of a line it will have 
zero surround in order to make the backend happy.)

In order to avoid the 'current' issue I'll use an overload of 
end_of_math in the generic code.

The real solution is to make the built-in end_of_math more strict 
(trivial patch ready and tested so that can end up in the next luatex).

In the context code i won't intercept it because end_of_math will do 
that at some point.

Hans

-----------------------------------------------------------------
                                           Hans Hagen | PRAGMA ADE
               Ridderstraat 27 | 8061 GH Hasselt | The Netherlands
     tel: 038 477 53 69 | voip: 087 875 68 74 | www.pragma-ade.com
                                              | www.pragma-pod.nl
-----------------------------------------------------------------
___________________________________________________________________________________
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-04-05 22:44 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-03 16:29 Dohyun Kim
2014-04-03 21:56 ` Hans Hagen
2014-04-05 13:12   ` Philipp Gesang
2014-04-05 22:44     ` Hans Hagen [this message]
2014-04-06  8:38       ` Philipp Gesang
2014-04-06 14:57       ` Philipp Gesang

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=53408759.9060302@wxs.nl \
    --to=pragma@wxs.nl \
    --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).