qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


From: Stefan Weil
Subject: Re: [Qemu-devel] Announcement: Patchew server is online now
Date: Tue, 03 Mar 2015 07:14:45 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.4.0

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/address@hidden

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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]