On 09/14/2016 08:33 PM, Greg KH wrote:
> On Wed, Sep 14, 2016 at 08:16:55PM +0200, Christian Borntraeger wrote:
>> On 09/14/2016 08:06 PM, Joe Perches wrote:
>>> On Wed, 2016-09-14 at 19:56 +0200, Christian Borntraeger wrote:
>>>
This will certainly help to reduce the noise. On the other hand
On Wed, Sep 14, 2016 at 05:05:09PM -0700, Joe Perches wrote:
> On Wed, 2016-09-14 at 16:54 -0700, Josh Triplett wrote:
> > On Wed, Sep 14, 2016 at 07:56:55PM +0200, Christian Borntraeger wrote:
> > > On 09/14/2016 07:51 PM, Joe Perches wrote:
> > > > checkpatch can be a useful tool for patches.
> >
On Wed, 2016-09-14 at 16:54 -0700, Josh Triplett wrote:
> On Wed, Sep 14, 2016 at 07:56:55PM +0200, Christian Borntraeger wrote:
> > On 09/14/2016 07:51 PM, Joe Perches wrote:
> > > checkpatch can be a useful tool for patches.
> > >
> > > It can be a much more controversial tool when used on files
On Wed, Sep 14, 2016 at 07:56:55PM +0200, Christian Borntraeger wrote:
> On 09/14/2016 07:51 PM, Joe Perches wrote:
> > checkpatch can be a useful tool for patches.
> >
> > It can be a much more controversial tool when used on files with the
> > -f option for style and whitespace changes for code
On Wed, 2016-09-14 at 20:54 +0200, Christian Borntraeger wrote:
> On 09/14/2016 08:33 PM, Greg KH wrote:
> > That's not true, I _WANT_ checkpatch cleanups for the portion of the
> > kernel I maintain. It keeps the code correct, up to date, easier to
> > maintain, and in doing so, we have found rea
On 09/14/2016 08:33 PM, Greg KH wrote:
> On Wed, Sep 14, 2016 at 08:16:55PM +0200, Christian Borntraeger wrote:
>> On 09/14/2016 08:06 PM, Joe Perches wrote:
>>> On Wed, 2016-09-14 at 19:56 +0200, Christian Borntraeger wrote:
>>>
This will certainly help to reduce the noise. On the other hand
On Wed, Sep 14, 2016 at 08:16:55PM +0200, Christian Borntraeger wrote:
> On 09/14/2016 08:06 PM, Joe Perches wrote:
> > On Wed, 2016-09-14 at 19:56 +0200, Christian Borntraeger wrote:
> >
> >> This will certainly help to reduce the noise. On the other hand I remember
> >> Linus
> >> saying someth
On 09/14/2016 08:21 PM, Joe Perches wrote:
> On Wed, 2016-09-14 at 20:16 +0200, Christian Borntraeger wrote:
>> You know what.
>> with some additional writing like
>> "Existing code outside staging is not supposed to be "fixed" to match
>> checkpatch.
>> Please do not send checkpatch initiated p
On Wed, 2016-09-14 at 20:16 +0200, Christian Borntraeger wrote:
> You know what.
> with some additional writing like
> "Existing code outside staging is not supposed to be "fixed" to match
> checkpatch.
> Please do not send checkpatch initiated patches for those files"
> near the newly created w
On 09/14/2016 08:06 PM, Joe Perches wrote:
> On Wed, 2016-09-14 at 19:56 +0200, Christian Borntraeger wrote:
>
>> This will certainly help to reduce the noise. On the other hand I remember
>> Linus
>> saying something along the line that he does not like the -f parameter (and
>> he
>> prefers to
On Wed, 2016-09-14 at 19:56 +0200, Christian Borntraeger wrote:
> This will certainly help to reduce the noise. On the other hand I remember
> Linus
> saying something along the line that he does not like the -f parameter (and he
> prefers to set this automatically). So while I like the approach
On 09/14/2016 07:51 PM, Joe Perches wrote:
> checkpatch can be a useful tool for patches.
>
> It can be a much more controversial tool when used on files with the
> -f option for style and whitespace changes for code that is relatively
> stable, obsolete, or for maintained by specific individuals.
12 matches
Mail list logo