[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