[PATCH] staging: rtl8723au: Fix brace coding style issues reported by checkpatch

Jeff Haran Jeff.Haran at citrix.com
Tue Oct 28 14:07:47 EDT 2014


> -----Original Message-----
> From: kernelnewbies-bounces+jharan=bytemobile.com at kernelnewbies.org [mailto:kernelnewbies-
> bounces+jharan=bytemobile.com at kernelnewbies.org] On Behalf Of Greg Donald
> Sent: Tuesday, October 28, 2014 10:39 AM
> To: Greg Freemyer
> Cc: kernelnewbies; Jeff Kirsher; Nicholas Krause
> Subject: Re: [PATCH] staging: rtl8723au: Fix brace coding style issues reported by checkpatch
> 
> On Tue, Oct 28, 2014 at 11:50 AM, Greg Freemyer <greg.freemyer at gmail.com> wrote:
> > Lots of violations
> > checkpatch finds are intentionally left in place because correcting
> > them makes the code less readable, not more readable.
> 
> Yeah, but there are still hundreds of thousands of checkpatch
> violations throughout the kernel that if fixed would actually improve
> readability.
> 
> Path            Errors  Warnings
> drivers         200979  361350
> arch            98791   142300
> sound           25938   31028
> include         13651   25598
> fs              96353   22483
> net             3185    19216
> lib             8055    6578
> tools           1126    3972
> kernel          656     3203
> security        47      1247
> mm              203     1186
> scripts         824     1168
> crypto          1144    1095
> block           196     656
> Documentation   97      259
> init            29      173
> virt            18      152
> samples         27      118
> ipc             2       77
> usr             17      20
> firmware        11      19
> 
> The WARNING "line over 80 characters" currently accounts for 216K of
> the total violations.  IMHO checkpatch should just stop complaining
> about the 80 char limit since that's the main offender causing new
> kernel developers to inadvertently lessen readability with their first
> patch.  If the 80 char warning should be mostly ignored why have it..
> it's pointless.  Increase it to a 21st century value or kill it.
> 
> 
> --
> Greg Donald

But, but, but, what about all the kernel developers who are writing kernel code on VT100s and storing their sources on 80 column punch cards?

Not everybody can afford these newfangled disk drives, ya know. 8^)

Jeff Haran




More information about the Kernelnewbies mailing list