From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54880) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bWoqN-0008FY-DA for qemu-devel@nongnu.org; Mon, 08 Aug 2016 14:00:08 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bWoqH-0000hi-Kp for qemu-devel@nongnu.org; Mon, 08 Aug 2016 14:00:06 -0400 Received: from mail-wm0-x235.google.com ([2a00:1450:400c:c09::235]:38065) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bWoqH-0000hZ-9n for qemu-devel@nongnu.org; Mon, 08 Aug 2016 14:00:01 -0400 Received: by mail-wm0-x235.google.com with SMTP id o80so155576341wme.1 for ; Mon, 08 Aug 2016 11:00:00 -0700 (PDT) MIME-Version: 1.0 References: <20160808141439.16908-1-marcandre.lureau@redhat.com> In-Reply-To: From: =?UTF-8?B?TWFyYy1BbmRyw6kgTHVyZWF1?= Date: Mon, 08 Aug 2016 17:59:48 +0000 Message-ID: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v3 00/15] qapi: remove the 'middle' mode List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: no-reply@patchew.org Cc: famz@redhat.com, qemu-devel@nongnu.org, armbru@redhat.com On Mon, Aug 8, 2016 at 6:57 PM wrote: > Hi, > > Your series seems to have some coding style problems. See output below fo= r > more information: > While at it, I guess I could remove those trailing spaces too > > Message-id: 20160808141439.16908-1-marcandre.lureau@redhat.com > Type: series > Subject: [Qemu-devel] [PATCH v3 00/15] qapi: remove the 'middle' mode > > =3D=3D=3D TEST SCRIPT BEGIN =3D=3D=3D > #!/bin/bash > > BASE=3Dbase > n=3D1 > total=3D$(git log --oneline $BASE.. | wc -l) > failed=3D0 > > commits=3D"$(git log --format=3D%H --reverse $BASE..)" > for c in $commits; do > echo "Checking PATCH $n/$total: $(git show --no-patch --format=3D%s > $c)..." > if ! git show $c --format=3Demail | ./scripts/checkpatch.pl --mailbac= k > -; then > failed=3D1 > echo > fi > n=3D$((n+1)) > done > > exit $failed > =3D=3D=3D TEST SCRIPT END =3D=3D=3D > > Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 > From https://github.com/patchew-project/qemu > * [new tag] patchew/ > 20160808141439.16908-1-marcandre.lureau@redhat.com -> patchew/ > 20160808141439.16908-1-marcandre.lureau@redhat.com > Switched to a new branch 'test' > 3879f40 qmp-commands.txt: fix some styling > f962de2 Drop qmp-commands.hx > 69cf17d build-sys: remove qmp-commands-old.h > fb249f9 monitor: use qmp_dispatch() > ce3a886 qmp: update qmp_query_spice fallback > e9443e9 qapi: check invalid arguments on no-args commands > 6159f41 qapi: remove the "middle" mode > 51a6744 monitor: implement 'qmp_query_commands' without qmp_cmds > 48e462f monitor: remove mhandler.cmd_new > 1705b2d monitor: register the qapi generated commands > 82316cd qapi: add 'export-marshal' command key > 3bd2e14 monitor: remove usage of generated marshal functions > f283f5e monitor: register gen:false commands manually > f9b1148 qapi-schema: add 'device_add' > 53e5fc1 qapi-schema: use generated marshaller for 'qmp_capabilities' > > =3D=3D=3D OUTPUT BEGIN =3D=3D=3D > Checking PATCH 1/15: qapi-schema: use generated marshaller for > 'qmp_capabilities'... > Checking PATCH 2/15: qapi-schema: add 'device_add'... > Checking PATCH 3/15: monitor: register gen:false commands manually... > Checking PATCH 4/15: monitor: remove usage of generated marshal > functions... > Checking PATCH 5/15: qapi: add 'export-marshal' command key... > Checking PATCH 6/15: monitor: register the qapi generated commands... > Checking PATCH 7/15: monitor: remove mhandler.cmd_new... > Checking PATCH 8/15: monitor: implement 'qmp_query_commands' without > qmp_cmds... > Checking PATCH 9/15: qapi: remove the "middle" mode... > Checking PATCH 10/15: qapi: check invalid arguments on no-args commands..= . > Checking PATCH 11/15: qmp: update qmp_query_spice fallback... > Checking PATCH 12/15: monitor: use qmp_dispatch()... > Checking PATCH 13/15: build-sys: remove qmp-commands-old.h... > Checking PATCH 14/15: Drop qmp-commands.hx... > ERROR: trailing whitespace > #5128: FILE: qmp-commands.txt:73: > +Arguments: $ > > ERROR: trailing whitespace > #6620: FILE: qmp-commands.txt:1565: > +Arguments: $ > > ERROR: trailing whitespace > #7346: FILE: qmp-commands.txt:2291: > + $ > > ERROR: trailing whitespace > #7363: FILE: qmp-commands.txt:2308: > + $ > > ERROR: trailing whitespace > #7747: FILE: qmp-commands.txt:2692: > + (such as RDMA pinning) which may be expensive, but do not= $ > > ERROR: trailing whitespace > #7748: FILE: qmp-commands.txt:2693: > + actually occur during the iterative migration rounds $ > > total: 6 errors, 0 warnings, 3887 lines checked > > Your patch has style problems, please review. If any of these errors > are false positives report them to the maintainer, see > CHECKPATCH in MAINTAINERS. > > Checking PATCH 15/15: qmp-commands.txt: fix some styling... > =3D=3D=3D OUTPUT END =3D=3D=3D > > Test command exited with code: 1 --=20 Marc-Andr=C3=A9 Lureau