Re: New driver: s3d2xfb

["Antonino A. Daplas" <adaplas@gmail.com>]

Richard Smith wrote:
>>>       if (!cnt) {
>>>               save_s3(info);
>>>       }
> 
>> You don't need to enclose a single statement with "curly braces".
> 
> I'm not trying to start any kind of flameage here but my curiosity was
> just tweaked by that statement.
> 
> What do you like about single line if's over ones with {} that leads
> you to advocate their usage?  You've obviously got loads of experience
> with a large amount of code written by other people so I'm interested
> in your opinion.

I don't really care which way, but it's the preferred, albeit,
undocumented kernel coding style.

Tony


-------------------------------------------------------
This SF.Net email is sponsored by:
Power Architecture Resource Center: Free content, downloads, discussions,
and more. http://solutions.newsforge.com/ibmarch.tmpl