Note that the 80 column rule has been relaxed for some time when
things are clearer when a little longer. Add in that things that people
grep for, such as error messages, shouldn't be broken up which is
the most common reason people exceed 80 columns intentionally.
Details
- Reviewers
melifaro jhb domagoj.stolfa_gmail.com - Group Reviewers
manpages - Commits
- rGa3238e92d704: style: Relax 80 column rule
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 39176 Build 36065: arc lint + arc unit
Event Timeline
I still use an 80 character limit, and an 80 character-wide Terminal window.
Usually my opinion means everyone else has moved on :).
Update to reflect discussion here and on IRC. Rather than
take a bold step to 120, just relax 80.
If we were to update clang-format to reflect the new rules, what value would we write there?
Personally I’d love to see something explicit so we leave all of the style(9) discussions to the standard formatter.
Keep it at 80 if you want clang-format to produce something that's acceptable. Also, clang-format doesn't have a 'don't break up long strings' rule that can be enabled last time I checked.
However, style(9) documents the range of acceptable and preferred styles in the tree. There isn't a canonical representation of style(9) for some code, as there's good reasons to vary a bit from any normal forms to improve the readability of the code. This is unlikely to change as there's too much code in the tree to reformat to 'one canonical style': the churn would cause more problems than it would solve.