<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Aug 23, 2019 at 9:11 AM Hans Hagen <<a href="mailto:j.hagen@xs4all.nl">j.hagen@xs4all.nl</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">On 8/23/2019 12:43 AM, Marcel Fabian Krüger wrote:<br>
> Hello,<br>
> <br>
> in tex_setmathparm and tex_getmathparm from ltexlib.c, `i` stores the<br>
> index of the math parameter to query or set. Given that `i` is obtained<br>
> through luaL_checkoption it should never have a invalid value. The<br>
> function still check that `i` is in the valid range using the check<br>
> <br>
>          if (i<0 && i>=math_param_last) {<br>
>              /* invalid spec, just ignore it  */<br>
> <br>
> I think this is a typo because i>=math_param_last (with<br>
> math_param_last=114) implies i>0 and therefore this condition can never<br>
> be true. Maybe || could be used instead of && or the check could be<br>
> removed entirly.<br>
in luametatex i use this test<br>
<br>
         if (i < 0 || j < 0) {<br>
<br>
so i'll do that in luatex too<br></blockquote><div><br></div><div> </div></div><div>Committed revision 7183.<br></div>-- <br><div dir="ltr" class="gmail_signature">luigi<br></div></div>