[PATCH] reformat_with_checkpatch: Add automation to checkpatch
From: joe@perches.com (Joe Perches)
Date: 2014-07-13 03:06:50
Also in:
kernel-janitors, lkml
From: joe@perches.com (Joe Perches)
Date: 2014-07-13 03:06:50
Also in:
kernel-janitors, lkml
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:quoted
On Sat, 2014-07-12 at 10:55 -0700, Greg KH wrote:quoted
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.