<div dir="ltr">Hi Karl,<div><br></div><div>Thanks so much for the follow up!<br><br>Eran</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 31, 2023 at 12:48 PM Karl Berry <<a href="mailto:karl@freefriends.org">karl@freefriends.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Eran - back on your report from August 2022 (sorry).<br>
<br>
    *page 297:* The index for "remainder" should link to section 104<br>
    ("remainder, scaled") instead of 543 (different "remainder" from<br>
    font-metric file)<br>
<br>
Agreed. Added at<br>
<a href="https://tug.org/texmfbug/newbug.html#B297remainder" rel="noreferrer" target="_blank">https://tug.org/texmfbug/newbug.html#B297remainder</a>.<br>
<br>
    *page 470, section 1127:* First line of align_error - should it<br>
    check for abs(align_state) > 1 instead of > 2? Since I assume you<br>
    want align_state to become zero by adding a single open or close<br>
    brace. I'm probably wrong.<br>
<br>
Here is DRF's analysis, concluding that no change is warranted. The fact<br>
that it's in the trip test is pretty persuasive.<br>
<br>
Let me know if questions/problems/changes ... --thanks, karl.<br>
<br>
<br>
Date: Mon, 16 Oct 2023 20:36:49 +0000 (UTC)<br>
From: David Fuchs<br>
To: Karl Berry<br>
Subject: B470: align_state<br>
<br>
OK, I'm going to go a little out on a limb on the align_state business,<br>
but I don't have high confidence level. Consider a basic alignment:<br>
<br>
\halign{\indent#\hfil&\hskip 1em#\hfil\cr<br>
A&B\cr<br>
CC&dd\cr<br>
EEEE&F\cr}<br>
<br>
Works fine.  Now, let's trigger align_error by changing "CC" to "C{C"<br>
which results in the expected "Missing } inserted" when TeX gets to the<br>
& (and then recovers OK with the default fix).  Good.  Now let's change<br>
to "C{{C" which gives a similar result, except that we get the "Missing<br>
} inserted" error twice, as you would expect.  Also good. <br>
<br>
But when we change to "C{{{C" we instead get the "Misplaced alignment<br>
tab character &" error message.<br>
<br>
My best guess is that the idea was that if you left out a closing brace<br>
by accident (so align_state is 1), or mis-typed a closing brace as an<br>
opening brace (so align_state is 2), it was deemed to be most likely<br>
that you had a simple case of messed up braces, and deserved a message<br>
that points in that direction.<br>
<br>
But if you're deeper than that into levels of unclosed left braces,<br>
something way worse might well be going on, and the warning should be<br>
more along the lines of indicating that you're deep in the weeds and not<br>
anywhere near an alignment (maybe you thought you were even out of<br>
it). And in such a situation, if you only got a zillion "Missing }<br>
inserted" messages, you may not quite have enough clues that it's due to<br>
alignment stuff (even though the Help text helps), and the more direct<br>
"Misplaced alignment tab character &" is more to the point.<br>
<br>
So, I don't know if that's the actual motivation for the test to be<br>
">2" and maybe there's a better explanation.  Somehow I don't think is a<br>
mistake, though; the trip test does hit it.  -drf<br>
</blockquote></div>