<p dir="ltr">True, but this is a build break for clang (because it's a tautology).</p>
<div class="gmail_quote">On Jul 19, 2015 9:59 PM, "Rob Landley" <<a href="mailto:rob@landley.net">rob@landley.net</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 07/18/2015 04:09 PM, enh wrote:<br>
> ping.<br>
><br>
> On Fri, May 15, 2015 at 4:38 PM, enh <<a href="mailto:enh@google.com">enh@google.com</a>> wrote:<br>
>> Fix ps.c build.<br>
>><br>
>> Found by clang...<br>
>><br>
>> external/toybox/toys/pending/ps.c:315:21: error: address of array<br>
>> 'field->title' will always evaluate to 'true'<br>
>> [-Werror,-Wpointer-bool-conversion]<br>
>>         if (!field->title) strcpy(field->title, typos[field->which]);<br>
>>             ~~~~~~~~^~~~~<br>
<br>
Indeed, real bug, but ps.c is only about 75% complete. It doesn't<br>
implement "ps ax" yet, or distinguish between dash vs non-dash options.<br>
(That's why I haven't tested the corner cases yet. This is probably the<br>
command that's going to force me to make a run as root aboriginal linux<br>
test case build control image, because I need to control the PID numbers<br>
to get reproducible results...)<br>
<br>
I'm writing up release notes as we speak. I've got a half-dozen pending<br>
commits but there's _always_ a half dozen pending commits, so I need to<br>
snapshot and move on and then have the next release more quickly.<br>
<br>
Rob<br>
</blockquote></div>