[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20180711142943.chjedee3m7v6yogh@mwanda>
Date: Wed, 11 Jul 2018 17:29:43 +0300
From: Dan Carpenter <dan.carpenter@...cle.com>
To: John Whitmore <arigead@...il.com>
Cc: Greg KH <gregkh@...uxfoundation.org>, devel@...verdev.osuosl.org,
kstewart@...uxfoundation.org, keescook@...omium.org,
aastha.gupta4104@...il.com, linux-kernel@...r.kernel.org,
John Whitmore <johnfwhitmore@...il.com>, pombredanne@...b.com,
tglx@...utronix.de
Subject: Re: [PATCH v2 3/8] staging:rtl8192u: Correct spacing before and
after parenthesis - Style
On Wed, Jul 11, 2018 at 03:27:11PM +0100, John Whitmore wrote:
> On Wed, Jul 11, 2018 at 04:22:50PM +0200, Greg KH wrote:
> > On Wed, Jul 11, 2018 at 03:09:37PM +0100, John Whitmore wrote:
> > > On Wed, Jul 11, 2018 at 01:13:49PM +0200, Greg KH wrote:
> > > > On Tue, Jul 10, 2018 at 04:05:18PM +0100, John Whitmore wrote:
> > > > > Corrected coding style issues aroung opening and closed parenthesis. Spaces, or blank line, removed from after '(' or before ')'
> > > >
> > > > Wrap your changelog text at 72 columns like your editor asked you to :(
> > > >
> > > > I'll fix it up this time...
> > >
> > > Oops sorry and thanks. I thought the magic number was 75 which I got from:
> > >
> > > https://www.kernel.org/doc/html/v4.16/process/submitting-patches.html
> >
> > 75 works as well :)
> >
>
> Well if I went over 75, which I knew about I'm very sorry I messed up. I'll
> keep an eye out in future. Might be using the wrong editor but let's not go
> there.
Here is the line:
Corrected coding style issues aroung opening and closed parenthesis. Spaces, or blank line, removed from after '(' or before ')'
It's 128 characters.
regards,
dan carpenter
Powered by blists - more mailing lists