[illumos-Developer] (updated) webrev: 1046 wcsxfrm(3C) crashes when len arg is 0
Gordon Ross
gordon.w.ross at gmail.com
Thu May 19 08:38:15 PDT 2011
On Thu, May 19, 2011 at 12:29 AM, Jason King <jason.brian.king at gmail.com> wrote:
> Need to stop doing things before I've read all my email :)
> A minor style nit, I thought it was preferred for non booleans to use (expr
> != 0) vs. (expr) or (expr == 0) vs (!expr)
> aside from that, LGTM
[...]
Do reviewers want me to change both the one I added
and the one ten lines earlier? (the one I copied:)
Might as well get both, I guess.
Assuming a "yes", I've updated the webrev:
http://www.yalms.org/cr/wcsxfrm/
More information about the Developer
mailing list