Re: [NTG-context] "true" units in LuaMetaTeX

2022-10-27 Thread Hans Hagen via ntg-context

On 10/27/2022 10:22 AM, Max Chernoff via ntg-context wrote:


Hi Hans,


Anyway, I'll check it. Is anyone still using this "true" actually?


I've been playing around with LuaMetaLaTeX/LuaMetaPlain recently

https://github.com/zauguin/luametalatex

and plain.tex uses truein for typesetting the footnote rule. It's trivial

to fix that specific case, but I'm finding 140 other files in TeX Live that
use the "true" dimensions, 69 of which that have been updated in the past 3
years. I doubt that anyone in ConTeXt uses the "true" dimensions though.
well, we can keep it as depricated feature (as there is no measurable 
overhead when it's not used) .. although i might come up with some plain 
mode in lmtx (when i'm really bored as it makes little sense) i'm not 
going to worry much about / cq. follow old-school usage and 
compatibility wrt dvi driver related features (i'm not even sure if 
luametatex makes sense for other macro packages but we'll see, these 
things happens outside my scope anyway)


Hans

-
  Hans Hagen | PRAGMA ADE
  Ridderstraat 27 | 8061 GH Hasselt | The Netherlands
   tel: 038 477 53 69 | www.pragma-ade.nl | 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 / https://www.ntg.nl/mailman/listinfo/ntg-context
webpage  : https://www.pragma-ade.nl / http://context.aanhet.net
archive  : https://bitbucket.org/phg/context-mirror/commits/
wiki : https://contextgarden.net
___


Re: [NTG-context] "true" units in LuaMetaTeX

2022-10-27 Thread Max Chernoff via ntg-context

Hi Hans,

> Anyway, I'll check it. Is anyone still using this "true" actually?

I've been playing around with LuaMetaLaTeX/LuaMetaPlain recently

   https://github.com/zauguin/luametalatex
   
and plain.tex uses truein for typesetting the footnote rule. It's trivial
to fix that specific case, but I'm finding 140 other files in TeX Live that
use the "true" dimensions, 69 of which that have been updated in the past 3
years. I doubt that anyone in ConTeXt uses the "true" dimensions though.

> I don't check github, it's mostly there for the compile farm and 
> intermediate testing (e.g. math stuff) so sending me a mail is ok. When 
> there is some serious issue a mwe to the list is good enough.

K, will do.

Thanks,
-- Max
___
If your question is of interest to others as well, please add an entry to the 
Wiki!

maillist : ntg-context@ntg.nl / https://www.ntg.nl/mailman/listinfo/ntg-context
webpage  : https://www.pragma-ade.nl / http://context.aanhet.net
archive  : https://bitbucket.org/phg/context-mirror/commits/
wiki : https://contextgarden.net
___


Re: [NTG-context] \autoinsertnextspace regression / discrepancy, mkiv to lmtx

2022-10-27 Thread Hans Hagen via ntg-context

On 10/26/2022 3:35 PM, Rik Kabel via ntg-context wrote:


On 2022-10-26 07:43, Hans Hagen via ntg-context wrote:

On 10/26/2022 1:52 AM, Rik Kabel via ntg-context wrote:
The following example produces different results when processed by 
LMTX and MkIV. In particular, a space between the X and Y is not 
auto-inserted by LMTX, while it is by MkIV.


    \starttexdefinition TEST #1
   #1\autoinsertnextspace
    \stoptexdefinition
    \tt
    \starttext
   \TEST{X} \emph{Y}
    \stoptext

The MkIV result is the correct result (in my opinion).

more a side effect ... we lookahead and \emph is not some character

I made a variants that does abetter job on that

\starttexdefinition TEST #1
    #1%
    \autoinsertedspace % subtle name change
\stoptexdefinition

but you have to wait till we update,

Hans



Thank you in advance for the upcoming fix.

Can you tell us (me) why you chose to create a new macro, 
\autoinsertedspace, instead of changing the code for 
\autoinsertnextspace in LMTX? Do you see a circumstance under which the 
current LMTX behavior of \autoinsertnextspace is desirable or required? 


compatibility .. it also uses a different mechanism for which that name 
suits better



Will the new macro be available under MkIV?

I suppose I can do that if needed,

Hans

-
  Hans Hagen | PRAGMA ADE
  Ridderstraat 27 | 8061 GH Hasselt | The Netherlands
   tel: 038 477 53 69 | www.pragma-ade.nl | 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 / https://www.ntg.nl/mailman/listinfo/ntg-context
webpage  : https://www.pragma-ade.nl / http://context.aanhet.net
archive  : https://bitbucket.org/phg/context-mirror/commits/
wiki : https://contextgarden.net
___


Re: [NTG-context] "true" units in LuaMetaTeX

2022-10-27 Thread Hans Hagen via ntg-context

On 10/27/2022 8:11 AM, Max Chernoff via ntg-context wrote:

Hi Hans,

The unit scanner in LuaMetaTeX fails for any of the "true" dimensions.

With this test file:

\starttext
\vrule width 1truein height 1pt depth 0pt\relax

\vrule width 1in height 1pt depth 0pt\relax

\stoptext

I get this output:

tex error   > tex error on line 2 in file ./lmtx-true.tex: Valid 
keyword expected, likely 'true'



\vrule width 1truei
n height 1pt depth 0pt\relax

1 \starttext

2 >>  \vrule width 1truein height 1pt depth 0pt\relax
3
4 \vrule width 1in height 1pt depth 0pt\relax
5 \stoptext
6

I was able to fix this with the patch at the end of the email.

(Also, how do you want patches sent? Via email to the mailing list,
emailed directly to you, as a pull request on GitHub, or something
else?)


In a way 'true' has been dropped becauswe it doesn't do anything and I 
will likely remove that check completely (as \mag is also gone, although 
the performance hit for scanning true is neglectable in luametatex).


Anyway, I'll check it. Is anyone still using this "true" actually?

I don't check github, it's mostly there for the compile farm and 
intermediate testing (e.g. math stuff) so sending me a mail is ok. When 
there is some serious issue a mwe to the list is good enough.


Hans




-
  Hans Hagen | PRAGMA ADE
  Ridderstraat 27 | 8061 GH Hasselt | The Netherlands
   tel: 038 477 53 69 | www.pragma-ade.nl | 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 / https://www.ntg.nl/mailman/listinfo/ntg-context
webpage  : https://www.pragma-ade.nl / http://context.aanhet.net
archive  : https://bitbucket.org/phg/context-mirror/commits/
wiki : https://contextgarden.net
___


[NTG-context] "true" units in LuaMetaTeX

2022-10-27 Thread Max Chernoff via ntg-context
Hi Hans,

The unit scanner in LuaMetaTeX fails for any of the "true" dimensions. 

With this test file:

   \starttext
   \vrule width 1truein height 1pt depth 0pt\relax
   
   \vrule width 1in height 1pt depth 0pt\relax
   \stoptext

I get this output:

   tex error   > tex error on line 2 in file ./lmtx-true.tex: Valid keyword 
expected, likely 'true'
   

   \vrule width 1truei
   n height 1pt depth 0pt\relax
   
   1 \starttext
   2 >>  \vrule width 1truein height 1pt depth 0pt\relax
   3 
   4 \vrule width 1in height 1pt depth 0pt\relax
   5 \stoptext
   6 

I was able to fix this with the patch at the end of the email.

(Also, how do you want patches sent? Via email to the mailing list,
emailed directly to you, as a pull request on GitHub, or something
else?)

Thanks,
-- Max

-- >8 --
From 69f186daa9b3bf53697e6edf1bd9b066d64b35fe Mon Sep 17 00:00:00 2001
From: Max Chernoff <49086429+gucci-on-fl...@users.noreply.github.com>
Date: Wed, 26 Oct 2022 23:45:19 -0600
Subject: [PATCH] Fix scanning of "true" units

---
 source/tex/texscanning.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/source/tex/texscanning.c b/source/tex/texscanning.c
index 7f78d80..98a9c41 100644
--- a/source/tex/texscanning.c
+++ b/source/tex/texscanning.c
@@ -2304,6 +2304,7 @@ typedef enum scanned_unit {
 
 static int tex_aux_scan_unit(halfword *num, halfword *denom, halfword *value, 
halfword *order)
 {
+  AGAIN:
 do {
 tex_get_x_token();
 } while (cur_cmd == spacer_cmd);
@@ -2327,7 +2328,6 @@ static int tex_aux_scan_unit(halfword *num, halfword 
*denom, halfword *value, ha
 goto BACK_TWO;
 }
 cur_cs = save_cur_cs;
-  AGAIN:
 switch (chrone) {
 case 'p': case 'P':
 switch (chrtwo) {
-- 
2.37.3


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

maillist : ntg-context@ntg.nl / https://www.ntg.nl/mailman/listinfo/ntg-context
webpage  : https://www.pragma-ade.nl / http://context.aanhet.net
archive  : https://bitbucket.org/phg/context-mirror/commits/
wiki : https://contextgarden.net
___