Messages in this thread | | | Date | Tue, 02 Oct 2012 14:46:03 -0400 (EDT) | Subject | Re: [PATCH 1/2] net: ethernet: clean out braces / old code (found via checkpatch) | From | David Miller <> |
| |
From: matthew@walster.org Date: Tue, 2 Oct 2012 12:41:44 +0100
> From: Matthew Walster <matthew@walster.org> > > Remove an old commented out piece of code. > Remove an if(true) statement. > Remove a set of braces that weren't strictly necessary. > > All found by running checkpatch.pl against the code. > > Signed-off-by: Matthew Walster <matthew@walster.org>
That comment and that unconditional if() are documentation.
Don't be an automaton and blindly make changes based upon checkpatch.pl output.
| |