[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 08/37] qapi/common.py: Remove python compatibility workaround
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 08/37] qapi/common.py: Remove python compatibility workaround |
Date: |
Wed, 16 Sep 2020 14:34:45 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
John Snow <jsnow@redhat.com> writes:
> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
> scripts/qapi/common.py | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/scripts/qapi/common.py b/scripts/qapi/common.py
> index ba35abea47..4fb265a8bf 100644
> --- a/scripts/qapi/common.py
> +++ b/scripts/qapi/common.py
> @@ -119,10 +119,7 @@ def cgen(code, **kwds):
> raw = code % kwds
> if indent_level:
> indent = genindent(indent_level)
> - # re.subn() lacks flags support before Python 2.7, use re.compile()
> - raw = re.subn(re.compile(r'^(?!(#|$))', re.MULTILINE),
> - indent, raw)
> - raw = raw[0]
> + raw, _ = re.subn(r'^(?!(#|$))', indent, raw, flags=re.MULTILINE)
> return re.sub(re.escape(eatspace) + r' *', '', raw)
I missed this one in my "qapi: Bye-bye Python 2" series.
Can we use re.sub() instead?
- Re: [PATCH 07/37] qapi: add pylintrc, (continued)
[PATCH 09/37] qapi/common.py: Add indent manager, John Snow, 2020/09/15
[PATCH 08/37] qapi/common.py: Remove python compatibility workaround, John Snow, 2020/09/15
- Re: [PATCH 08/37] qapi/common.py: Remove python compatibility workaround,
Markus Armbruster <=
[PATCH 10/37] qapi/common.py: delint with pylint, John Snow, 2020/09/15
[PATCH 12/37] qapi/common.py: check with pylint, John Snow, 2020/09/15
[PATCH 11/37] qapi/common.py: Replace one-letter 'c' variable, John Snow, 2020/09/15
[PATCH 14/37] qapi/common.py: Move comments into docstrings, John Snow, 2020/09/15