[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 12/21] qapi.py: fix line break before binary
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH v7 12/21] qapi.py: fix line break before binary operator pep8 |
Date: |
Tue, 10 Jan 2017 07:55:15 -0500 (EST) |
Hi
----- Original Message -----
> Marc-André Lureau <address@hidden> writes:
>
> > Python code style accepts both form, but pep8 complains. Better to clean
> > up the single warning for now, so new errors stand out more easily.
> >
> > Fix scripts/qapi.py:1539:21: W503 line break before binary operator
>
> PEP 8 permits both, but prefers line break before binary operator.
> Thus, this warning is clearly bogus. What version of pep8 do you use?
> Mine doesn't warn.
I tried both 1.6.2 and 1.7.0
- [Qemu-devel] [PATCH v7 06/21] qga/schema: improve guest-set-vcpus Returns: section, (continued)
- [Qemu-devel] [PATCH v7 06/21] qga/schema: improve guest-set-vcpus Returns: section, Marc-André Lureau, 2017/01/09
- [Qemu-devel] [PATCH v7 05/21] qapi: improve TransactionAction doc, Marc-André Lureau, 2017/01/09
- [Qemu-devel] [PATCH v7 07/21] qapi: avoid interleaving sections and parameters, Marc-André Lureau, 2017/01/09
- [Qemu-devel] [PATCH v7 08/21] qapi: move experimental note down, Marc-André Lureau, 2017/01/09
- [Qemu-devel] [PATCH v7 09/21] qapi: add some sections in docs, Marc-André Lureau, 2017/01/09
- [Qemu-devel] [PATCH v7 12/21] qapi.py: fix line break before binary operator pep8, Marc-André Lureau, 2017/01/09
[Qemu-devel] [PATCH v7 13/21] texi2pod: learn quotation, deftp and deftypefn, Marc-André Lureau, 2017/01/09
[Qemu-devel] [PATCH v7 10/21] docs: add master qapi texi files, Marc-André Lureau, 2017/01/09
[Qemu-devel] [PATCH v7 11/21] qapi: rework qapi Exception, Marc-André Lureau, 2017/01/09
[Qemu-devel] [PATCH v7 20/21] build-sys: add txt documentation rules, Marc-André Lureau, 2017/01/09
[Qemu-devel] [PATCH v7 19/21] build-sys: use a generic TEXI2MAN rule, Marc-André Lureau, 2017/01/09
[Qemu-devel] [PATCH v7 17/21] build-sys: use --no-split for info, Marc-André Lureau, 2017/01/09
[Qemu-devel] [PATCH v7 16/21] docs: add qemu logo to pdf, Marc-André Lureau, 2017/01/09