[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