Messages in this thread | | | Date | Sun, 23 Mar 2008 05:19:29 -0700 (PDT) | Subject | Re: [PATCH 109/148] include/asm-x86/serial.h: checkpatch cleanups - formatting only | From | David Miller <> |
| |
From: Jiri Slaby <jirislaby@gmail.com> Date: Sun, 23 Mar 2008 13:06:04 +0100
> On 03/23/2008 11:20 AM, David Miller wrote: > > From: Al Viro <viro@ZenIV.linux.org.uk> > > Date: Sun, 23 Mar 2008 08:52:10 +0000 > > > >> On Sun, Mar 23, 2008 at 01:03:24AM -0700, Joe Perches wrote: > >>> Signed-off-by: Joe Perches <joe@perches.com> > >> An obvious corollary to this highly inspired series of patches: > > > > Acked-by: David S. Miller <davem@davemloft.net> > > > > I totally agree, checkpatch is just shit. > > I disagree. It's just misuse in this case (like using Lindent on whole tree).
Unlike sparse, this thing encourages the kind of behavior seen here.
And even worse it becomes monkey see monkey do.
There are mountains of more useful stuff to be working on (much of it automated, but unlike checkpatch work doesn't result in crap) rather than 148 patches of checkpatch vomit.
Fixing sparse warnings properly fixes real issues, whereas fixing checkpatch stuff creates garbage 9 times out of 10.
| |