Thread (11 messages) 11 messages, 4 authors, 2012-10-04
STALE4994d

[PATCH] net: ethernet: clean out braces / old code (found via checkpatch)

From: Scott Lovenberg <hidden>
Date: 2012-10-03 13:12:11

On Wed, Oct 3, 2012 at 5:13 AM, Matthew Walster [off-list ref] wrote:

On 2 October 2012 17:16, Scott Lovenberg [off-list ref] wrote:
quoted
Looks good to me.

Maintainer didn't think so :(

On 2 October 2012 19:46, David Miller [off-list ref] wrote:
quoted
That comment and that unconditional if() are documentation.

Don't be an automaton and blindly make changes based upon
checkpatch.pl output.

Perhaps I'll just clean up some of drivers/staging while I learn the process
before I dive in to "net" again.

Matthew Walster
Sorry, man.  I'm not going to mix it up with Dave Miller, but really
if he wanted that to stay in there a comment suggesting so would have
been nice.

-- 
Peace and Blessings,
-Scott.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help