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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





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

Maintainer didn't think so :(

On 2 October 2012 19:46, David Miller <davem@xxxxxxxxxxxxx> 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
_______________________________________________
Kernelnewbies mailing list
Kernelnewbies@xxxxxxxxxxxxxxxxx
http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies

[Index of Archives]     [Newbies FAQ]     [Linux Kernel Mentors]     [Linux Kernel Development]     [IETF Annouce]     [Git]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux SCSI]     [Linux ACPI]
  Powered by Linux