Thread (1 message) 1 message, 1 author, 2012-02-22

RE: [PATCH] checkpatch: Add some --strict coding style checks

From: Johannes Berg <johannes@sipsolutions.net>
Date: 2012-02-22 09:46:34
Also in: linux-wireless, lkml

On Wed, 2012-02-22 at 09:35 +0000, David Laight wrote:
quoted
	if (!(func_a(x) &&
		func_b(y)))
		baz();
Gah - that is horrid for the 'preferred style'.
A quick glance at the code puts both the func_b()
and baz() calls as inside the if.
You forgot to read/quote the rest of Allan's email:
Assuming my stupid mailer will screw up the indentation above, the 'a'
in addr in the first example is meant to be immediately below the 'n'
in nr, and the two 'f's in func_* are meant to be vertically lined up
in the second example.
johannes
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help