<div dir="ltr">no, that would be if you were calling fflush(). xflush() takes a bool.<div><br></div><div>being explicit with NULL or true -- which i do anyway, because life is confusing enough as it is -- wouldn't have stopped C from just implicitly convert everything anyway, but maybe xflush() should be broken in two (and/or changed to actually take a FILE*), with the "no, don't really flush" being private to xwrap --- i only see one caller outside of xwrap.c and that's sh.c and i think that's actually another instance of this bug?<br><div><br></div><div>anyway, a minimal patch that fixes grep sent to mailing list.</div><div><br></div><div>(this is github being helpful, btw --- i haven't even tried to sync AOSP yet. i'm just looking at the commits with the red X next to them on <a href="https://github.com/landley/toybox/commits/master">https://github.com/landley/toybox/commits/master</a>. )</div><div><div><br></div><div><br></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Aug 19, 2020 at 11:59 PM 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/19/20 7:33 PM, enh via Toybox wrote:<br>
> <br>
> This is why the tests have been failing since<br>
> 42303209f44a335025b9cd1dbe5dd2f3069f2e99.<br>
<br>
Blah. That first 0 is NULL, not stdin. (Mixing ansi stdio.h and filehandle<br>
contexts because it flushes one so the other is safe to do...)<br>
<br>
Rob<br>
</blockquote></div>