On 09/02/2012 01:51 PM, Blue Swirl wrote: > I've used the following snippet to check what happens with the last 100 > commits: > for i in '' 1 2 3 4 5 6 7 8 9; do for j in 0 1 2 3 4 5 6 7 8 9; do > echo $i$j; git show HEAD~$i$j >/tmp/a; clear;head -20 /tmp/a; > ./scripts/checkpatch.pl --root=. /tmp/a; read foo; done; done > > Sadly, it also shows how much stuff gets committed without checking, > and on the other hand, the amount of false alarms.
Maintainers should add a checkpatch invocation as a git hook (advisory only), this could reduce the amount of violations getting into the tree. -- error compiling committee.c: too many arguments to function