<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Sep 1, 2021 at 9:15 AM Rob Landley <<a href="mailto:rob@landley.net">rob@landley.net</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/30/21 4:00 PM, enh wrote:<br>
>     Your commit changed list_signals() to start from 1 but didn't do the same for<br>
>     xsignal_all_killers(). One character fix. <br>
> <br>
> yeah, i sent you a more complicated fix in case someone was tempted to reorder<br>
> the array (note that in list_signals() the loop variable is the signal number,<br>
> but in xsignal_all_killers() the loop variable is the array index), but (a) your<br>
> smaller fix works and (b) we know we have a test that will catch this if someone<br>
> does break it, so lgtm :-)<br>
<br>
It would break list_signals() if they reordered the list, so I didn't add a new<br>
dependency. :)<br></blockquote><div><br></div><div>no, it's cleverer than that... that's what fooled me into not looking for code that directly accesses the array --- list_signals() iterates through _signal numbers_ and asks num_to_sig() to search the table for each one:</div><div><br></div><div>  for (; i<=NSIG; i++) {<br>    if ((name = num_to_sig(i))) {<br>      printf("%2d) SIG%-9s", i, name);<br>      if (++count % cols == 0) putchar('\n');<br>    }<br>  }<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> thanks! (and sorry for not running all the tests! it's _just_ slow enough on my<br>
> laptop to run them all that i tend to only run the specific toy's tests.)<br>
<br>
Eh, I should have done it here myself. (Touching an individual command file<br>
doesn't break other commands, but touching lib/ or main.c does.)<br>
<br>
Rob<br>
</blockquote></div></div>