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

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

 



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.

Attachment: pgpAxRokck5Op.pgp
Description: PGP signature

_______________________________________________
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