[PATCH 2/3] staging: lustre: fix pointer whitespace style
Dan Carpenter
dan.carpenter at oracle.com
Mon Sep 8 10:32:08 UTC 2014
On Sat, Sep 06, 2014 at 11:19:03AM -0700, Joe Perches wrote:
>
> Indexing a string array with an enum is generally unsafe.
>
> I think it better to use a switch/case like:
>
The reverse side of that argument is that switch statements are slower
and uglier.
My understanding is that according to the c spec, "state" could be
signed but in this case GCC defines it as unsigned int. So the ASSERT
is ok in real life, but in theory it should check for negatives as well.
regards,
dan carpenter
More information about the devel
mailing list