Messages in this thread | | | Date | Wed, 30 Jul 2014 16:41:53 +0300 | From | Dan Carpenter <> | Subject | Re: [PATCH 13/19 v2] staging: comedi: amplc_pci230: remove unnecessary braces |
| |
On Wed, Jul 30, 2014 at 12:53:47PM +0100, Ian Abbott wrote: > Signed-off-by: Ian Abbott <abbotti@mev.co.uk> > --- > v2: Be less aggressive in brace removal. Keep them for outer nested > 'if's. Keep them in outer control statements where the inner control > statement uses braces.
Really every multi-line indent should have braces for readability. This isn't documented anywhere, but it's normal kernel style and Greg prefers it for staging as well but I can't find the URL on google where he mentioned that.
regards, dan carpenter
| |