[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