[PATCH] reformat_with_checkpatch: Add automation to checkpatch
Joe Perches
joe at perches.com
Sat Jul 12 23:06:36 EDT 2014
On Sat, 2014-07-12 at 12:28 -0700, Greg KH wrote:
> On Sat, Jul 12, 2014 at 11:29:37AM -0700, Joe Perches wrote:
> > On Sat, 2014-07-12 at 10:55 -0700, Greg KH wrote:
> > > Yes, the warnings are all due to line-length, but Joe, you shouldn't add
> > > a patch that causes more checkpatch warnings than before :)
> >
> > Yeah, that was intentional though.
> >
> > This script does a series of relatively discrete changes.
> >
> > Lindent would more or less work, but it's _horrible_ at
> > wrapping overlong lines and merges all types of changes
> > together.
>
> Oh I agree, I don't want to see Lindent, but maybe, if the patch adds
> checkpatch warnings, it should be at least "flagged" as maybe a problem?
The script I submitted does run checkpatch
before any commit is done.
> Would you want to be on the receiving end of this patch script?
You might have noticed I don't have a kernel.org
account.
More information about the Kernelnewbies
mailing list