Re: [Qemu-devel] [PATCH v2 0/4] CHECKPATCH: Add warning for single else statement.

2012-09-08 Thread Blue Swirl
On Thu, Sep 6, 2012 at 10:32 AM, Andreas Färber wrote: > Am 05.09.2012 21:46, schrieb Blue Swirl: >> Thanks, applied all. > > Question: Are additions to checkpatch.pl supposed to be in QEMU Coding > Style as done here? Do you plan to convert it consistently then? > checkpatch.pl uses tabs consiste

Re: [Qemu-devel] [PATCH v2 0/4] CHECKPATCH: Add warning for single else statement.

2012-09-06 Thread Andreas Färber
Am 05.09.2012 21:46, schrieb Blue Swirl: > Thanks, applied all. Question: Are additions to checkpatch.pl supposed to be in QEMU Coding Style as done here? Do you plan to convert it consistently then? checkpatch.pl uses tabs consistently, making checkpatch.pl complain. ;) The alternative would be t

Re: [Qemu-devel] [PATCH v2 0/4] CHECKPATCH: Add warning for single else statement.

2012-09-05 Thread Blue Swirl
Thanks, applied all. On Sun, Sep 2, 2012 at 11:22 PM, Don Slutz wrote: > Also add more debug options to find this issue. They were not listed > in the help because the are not simple to understand the output of. > > Using: > > rm zz; > for k in 0 1 2 3; do >for i in 0 1 2 3 4 5 6 7 8 9;

[Qemu-devel] [PATCH v2 0/4] CHECKPATCH: Add warning for single else statement.

2012-09-02 Thread Don Slutz
Also add more debug options to find this issue. They were not listed in the help because the are not simple to understand the output of. Using: rm zz; for k in 0 1 2 3; do for i in 0 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 $k$i$j; git show HEAD~$k$i$j >/t