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

Valdis.Kletnieks at vt.edu Valdis.Kletnieks at vt.edu
Tue Oct 28 14:42:16 EDT 2014


On Tue, 28 Oct 2014 12:39:17 -0500, Greg Donald said:

> 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

On the other hand, there's very good human factors reasons to restrict the
length of source lines.  As the human eye reads text, it saccades across,
and too-long lines result in poor tracking.  More than 60-75 characters
causes problems.

http://en.wikipedia.org/wiki/Eye_movement_in_reading
http://baymard.com/blog/line-length-readability

The fact that C code is indented doesn't help - it means the left
margin isn't as easily predictable as flush-left alignment.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 848 bytes
Desc: not available
Url : http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20141028/8e6cb03b/attachment.bin 


More information about the Kernelnewbies mailing list