Re: [Qemu-devel] [PATCH v2] scripts/checkpatch.pl: Bug fix

2018-04-03 Thread Su Hang
Please ignore this email, this patch was accidentally sent. "Su Hang" wrote: > checkpatch.pl stops complaining about following pattern: > """ > do { > //do somethins; > } while (conditions); > """ > > One things need to be mentioned: > Becasue `if`, `while` and `for` check have been done in t

[Qemu-devel] [PATCH v2] scripts/checkpatch.pl: Bug fix

2018-04-03 Thread Su Hang
checkpatch.pl stops complaining about following pattern: """ do { //do somethins; } while (conditions); """ One things need to be mentioned: Becasue `if`, `while` and `for` check have been done in this `if` block(Line: 2356), and this block contains following statement: """ Line: 2379 $suppres

Re: [Qemu-devel] [PATCH v2] scripts/checkpatch.pl: Bug fix

2018-03-16 Thread Su Hang
el@nongnu.org > Subject: Re: [Qemu-devel] [PATCH v2] scripts/checkpatch.pl: Bug fix > > On Fri, 03/16 14:31, Su Hang wrote: > > Because I generate my first patch on master, it should work. > > Nope. "git am" doesn't know to resolve the conflict automatically des

Re: [Qemu-devel] [PATCH v2] scripts/checkpatch.pl: Bug fix

2018-03-16 Thread Fam Zheng
On Fri, 03/16 14:31, Su Hang wrote: > Because I generate my first patch on master, it should work. Nope. "git am" doesn't know to resolve the conflict automatically despite it being straightforward, unfortunately: http://patchew.org/QEMU/1521169105-32041-1-git-send-email-suhan...@mails.ucas.ac.cn

Re: [Qemu-devel] [PATCH v2] scripts/checkpatch.pl: Bug fix

2018-03-15 Thread Su Hang
ot; > Sent Time: 2018-03-16 14:15:20 (Friday) > To: "Su Hang" > Cc: peter.mayd...@linaro.org, ebl...@redhat.com, vsement...@virtuozzo.com, > qemu-devel@nongnu.org > Subject: Re: [Qemu-devel] [PATCH v2] scripts/checkpatch.pl: Bug fix > > On Fri, 03/16 10:58, Su

Re: [Qemu-devel] [PATCH v2] scripts/checkpatch.pl: Bug fix

2018-03-15 Thread Fam Zheng
On Fri, 03/16 10:58, Su Hang wrote: > (Sorry, I don't know this patch should base on which commit, > so I generate this patch based on > commit:fb8446d94ec7a3dc0c3a7e7da672406476f075ac, > I choose this by `git log -2 scripts/checkpath.pl`. > Sincerely say sorry, if I have misunderstand any mean

[Qemu-devel] [PATCH v2] scripts/checkpatch.pl: Bug fix

2018-03-15 Thread Su Hang
checkpatch.pl stops complaining about following pattern: """ do { //do somethins; } while (conditions); """ One things need to be mentioned: Becasue `if`, `while` and `for` check have been done in this `if` block(Line: 2356), and this block contains following statement: """ Line: 2379 $suppres