Messages in this thread | | | Subject | Re: [PATCH] .clang-format: update column limit | From | Joe Perches <> | Date | Thu, 11 Jun 2020 03:36:34 -0700 |
| |
On Thu, 2020-06-11 at 12:03 +0200, Miguel Ojeda wrote: > Hi Joe, > > On Wed, Jun 10, 2020 at 7:13 PM Joe Perches <joe@perches.com> wrote: > > Ii think this is a not a good change. > > > > If you read the commit log you provided, it ways > > "staying withing 80 columns is certainly still _preferred_" > > Yes, but the related email discussions were not about establishing a > new hard limit, but about avoiding such hard limits for > historical/technical reasons.
Exactly. So don't set a new hard limit of 100.
This would _always_ wrap lines to 100 columns when 80 columns is still preferred.
Imagine using a 100 column limit where a statement still fits on 2 lines. Now imagine the same statement wrapped at 80 columns still fitting on 2 lines.
Which would you prefer and why?
| |