Off the top of my head, I'd say the effective 77-char restriction is just a
bug due to not considering the newline char.
The motivation for <80 char is so that you can look at diffs/patches in an
80-char window without wrapping. If you only ever use context diffs, then
a 79-char limit would be sufficient, but for normal diffs ('> '/'< '
prefix) then it needs to be 78. I don't think it has anything to do with
emacs.
Steve
On Wed, Mar 16, 2016 at 9:22 AM Andreas Sandberg <[email protected]>
wrote:
> Hi Everyone,
>
> I recently noticed that the style checker forces lines to be 77 characters
> or shorter and not 78 characters. This is presumably caused by the checker
> not stripping the newline character from the line when calculating the
> length. Is this intentional? The style guide seems to mandate 78
> characters or less.
>
> I¹d also like to know what the reasoning behind 78 characters and why it
> isn¹t 80 characters. If it¹s about control characters (e.g., Emacs¹s line
> continuation character), 79 characters should work equally well. Judging
> by the code in validate() in style.py, the style guide used to mandate 79
> characters or less (excluding newline).
>
> //Andreas
>
> IMPORTANT NOTICE: The contents of this email and any attachments are
> confidential and may also be privileged. If you are not the intended
> recipient, please notify the sender immediately and do not disclose the
> contents to any other person, use it for any purpose, or store or copy the
> information in any medium. Thank you.
>
> _______________________________________________
> gem5-dev mailing list
> [email protected]
> http://m5sim.org/mailman/listinfo/gem5-dev
>
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
