Re: [ros-dev] [ros-diffs] [dgorbachev] 39422: Fix bug #4129 in CmpGetNameControlBlock().

[Aleksey Bragin]

Good, let's discuss it developers-wide then, and add a result into  
the (to become official) Coding Style Guidelines for kernel-mode code.
The only thing I'm bothered with is to keep the style consistent  
across kernel (and drivers, where possible). And right now bracing  
arguments of && and || prevails.

I'm eager to listen to other dev's opinion on that topic. You just  
showed one disadvantage of such method. Are there any advantages?


WBR,
Aleksey.

On Feb 16, 2009, at 8:47 PM, Dmitry Gorbachev wrote:

> >   No, it's
> > just a matter of coding style used in the kernel. 
>  What is a logical basis for adopting this particular rule?  
>  Sometimes it is
>  not just a matter of preference.
>
>  These parentheses are unnecessary for humans, and if they are not  
>  needed for
>  compilers or some other tools, they are bad! Without them, the bug  
>  could be
>  noticed earlier.
>
>  Cheers,
>  Dmitry