[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] CHECKPATCH
From: |
Avi Kivity |
Subject: |
Re: [Qemu-devel] [PATCH] CHECKPATCH |
Date: |
Thu, 06 Sep 2012 18:32:26 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:14.0) Gecko/20120717 Thunderbird/14.0 |
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
- [Qemu-devel] [PATCH] CHECKPATCH, Don Slutz, 2012/09/01
- [Qemu-devel] [PATCH v2 2/4] CHECKPATCH: Add --debug adv_checking, Don Slutz, 2012/09/02
- [Qemu-devel] [PATCH v2 1/4] CHECKPATCH: Add --debug adv_dcs, Don Slutz, 2012/09/02
- [Qemu-devel] [PATCH v2 4/4] CHECKPATCH: Add warning for single else statement., Don Slutz, 2012/09/02
- [Qemu-devel] [PATCH v2 3/4] CHECKPATCH: Add --debug adv_apw, Don Slutz, 2012/09/02