Thread (27 messages) 27 messages, 5 authors, 2014-10-30

[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

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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help