Re: [Qemu-devel] Announcement: Patchew server is online now

[Stefan Weil <sw@weilnetz.de>]

Am 28.02.2015 um 07:47 schrieb Fam Zheng:
> Hello, everyone
>
> I'm glad to announce the fresh Patchew server deployment:
>
>      http://qemu.patchew.org/
>
> The mission of this project is to help identify obvious defects (coding style,
> compiling, etc.) of posted patches.
>
> Follow the green "passed" or red "failed" button in the page and you'll see the
> testing log. You can also see which series have got reviews or comments.
>
> Q: How will this impact patch submission and merging process?
>
>      Nothing is affected (at least for now), use this as an auxiliary.
>
> Q: What exactly are tested for each patch?
>
>      * scripts/checkpatch.pl
>      * git am (on top of qemu.git)

Obviously checkpatch.pl reports false positives because it tries to 
analyze quoted-printable UTF-8 e-mails. Here an example:

http://qemu.patchew.org/testing/log/%3C1425214326-25326-1-git-send-email-sw@weilnetz.de%3E

Here is the message from Patchew:

<<< WARNING >>>ERROR: spaces required around that '=' (ctx:WxV)
<<< WARNING >>>#96: FILE: util/oslib-posix.c:404:
<<< WARNING >>>+        for (i =3D 0; i < numpages; i++) {
<<< WARNING >>>                ^

It's wrong because "=3D" is simply a quoted "=", and it is surrounded by 
spaces here.

Regards
Stefan