[PATCH 2/3] staging: lustre: fix pointer whitespace style

Joe Perches joe at perches.com
Mon Sep 8 11:37:44 UTC 2014


On Mon, 2014-09-08 at 13:32 +0300, Dan Carpenter wrote:
> 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.

Uglier is true, safer is true too.
Slower depends.
Sometimes the actual code is identical.




More information about the devel mailing list