Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Alex Dewar <alex.dewar90@gmail.com>
Date:   Tue, 18 Aug 2020 17:53:59 +0100
To:     Joe Perches <joe@perches.com>
Cc:     Alex Dewar <alex.dewar90@gmail.com>,
        Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>,
        linux-kernel@vger.kernel.org, clang-built-linux@googlegroups.com
Subject: Re: [PATCH] clang-format: Fix: Maximum line length is now 100
Message-ID: <20200818165359.jye5ch4uvdvkil36@medion>
References: <20200818162431.696423-1-alex.dewar90@gmail.com>
 <15980e07c369b2cbfc7b06f58f3a521a5495bdf6.camel@perches.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <15980e07c369b2cbfc7b06f58f3a521a5495bdf6.camel@perches.com>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Tue, Aug 18, 2020 at 09:52:20AM -0700, Joe Perches wrote:
> On Tue, 2020-08-18 at 17:24 +0100, Alex Dewar wrote:
> > Since commit bdc48fa11e46 ("checkpatch/coding-style: deprecate 80-column
> > warning") the kernel style is now for lines to be a maximum of 100 rather
> > than 80 columns. Update .clang-format accordingly.
> []
> > diff --git a/.clang-format b/.clang-format
> []
> > @@ -52,7 +52,7 @@ BreakConstructorInitializersBeforeComma: false
> >  #BreakConstructorInitializers: BeforeComma # Unknown to clang-format-4.0
> >  BreakAfterJavaFieldAnnotations: false
> >  BreakStringLiterals: false
> > -ColumnLimit: 80
> > +ColumnLimit: 100
> 
> No thank you.
> 
> 100 is used for exceptional line length, not
> for reformatting all lines up to 100 chars.
> 
> See the previous discussion about the same patch
> submitted earlier in June:
> 
> https://lore.kernel.org/lkml/20200610125147.2782142-1-christian.brauner@ubuntu.com/
> 
> 

Ahh, my mistake. Thanks for getting back to me anyhoo.

Alex