|
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
[Prev in Thread] | Current Thread | [Next in Thread] |