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-12 01:50:45
Also in: kernel-janitors, lkml

On Fri, 2014-07-11 at 18:43 -0700, Greg KH wrote:
On Fri, Jul 11, 2014 at 06:21:27PM -0700, Joe Perches wrote:
quoted
A simple script to run checkpatch --fix for various types of
of cleanups.
[]
If I pick drivers/staging/lustre/include/linux/lnet/types.h, then I get:

	staging: lnet: checkpatch cleanup: whitespace neatening

and no 'types.h' here, is that intentional?  If so, why?
Yes, it's how it's written.
It uses just the directory name, not any basename($file)

It can be changed if that's what's desired.
And this is fun, I'm going to let this rip on the lustre code...
It doesn't autocommit, it does show the various changes
it makes and asks you to accept them.

I suppose that could be automated with something better
than "yes", but I didn't want it to be completely automatic.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help