On Wednesday, July 30, 2014 7:28 AM, Ian Abbott wrote: > On 2014-07-30 15:09, Dan Carpenter wrote: >> On Wed, Jul 30, 2014 at 03:04:03PM +0100, Ian Abbott wrote: >>> On 2014-07-30 14:41, Dan Carpenter wrote: >>>> On Wed, Jul 30, 2014 at 12:53:47PM +0100, Ian Abbott wrote: >>>>> Signed-off-by: Ian Abbott <[email protected]> >>>>> --- >>>>> 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. >>> >>> With this version of the patch, amplc_pci230.c conforms to that -- >>> as long as you don't count comments, or splitting a simple statement >>> across multiple lines to fit within 80 columns, e.g.: >>> >>> if (foo) >>> /* do this */ >>> bar = baz + >>> qux; >> >> Heh... I am counting comments and broken lines, though. It's not a >> totally strict rule and it's especially not strict for broken lines. >> But we would prefer braces in those situations. > > If necessary, patches 13 and 19 can be dropped entirely.
That might be best for now. The rest of the series applies fine with them dropped. Regards, Hartley -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
