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

Matthew Walster matthew at walster.org
Wed Oct 3 05:13:48 EDT 2012


On 2 October 2012 17:16, Scott Lovenberg <scott.lovenberg at gmail.com> wrote:
>
> Looks good to me.
>

Maintainer didn't think so :(

On 2 October 2012 19:46, David Miller <davem at davemloft.net> wrote:
>
> 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20121003/8cc33af7/attachment.html 


More information about the Kernelnewbies mailing list