[PATCH] ASoC: Clean up coding style issues in GTA02

Nicolas Dufresne nicolas.dufresne at gmail.com
Sat Apr 4 16:04:53 CEST 2009


What it means is that you should not mix like:

        if (val) {
        	do_stuff();
        	do_more_stuff();
        }
        else
        	do_other_suff();

All the rest is user choice, Not changing it for fun will reduce noise
into you patches and reviewer will better manage to focus on what you
really are trying to fix.

But that's my opinion,
Nicolas

Le vendredi 03 avril 2009 à 18:43 +0200, pHilipp Zabel a écrit :
> | This does not apply if ONE branch of a conditional statement is a
> single
> | statement. Use braces in both branches.
> 
> I read this as meaning that if BOTH branches of a conditional
> statement are a single statement, not using braces is still preferred.




More information about the openmoko-kernel mailing list