From: Jiri Slaby <jirislaby@xxxxxxxxx>
Date: Sun, 23 Mar 2008 13:24:23 +0100
On 03/23/2008 01:19 PM, David Miller wrote:There are mountains of more useful stuff to be working on (much of itYes, I agree with you in this.
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.
What I don't agree with is that it's useless. It may help track down some issues in yet prepared patch (it's checkpatch, not checkcode and it should be used in that manner).
I strongly disagree still.
Half the warnings I get when I have run checkpatch on things I've
written were crap.
It's an automaton, and that's why people like it. It tells you
exactly what to do, and people like to be able to turn their brains
off like that.