[ros-dev] [ros-diffs] [dgorbachev] 39422: Fix bug #4129 in CmpGetNameControlBlock().
aleksey at reactos.org
Mon Feb 16 19:27:25 CET 2009
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?
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.
More information about the Ros-dev