[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/5] qapi: Drop Indentation.__bool__()
From: |
Eric Blake |
Subject: |
Re: [PATCH 2/5] qapi: Drop Indentation.__bool__() |
Date: |
Mon, 13 Sep 2021 13:29:17 -0500 |
User-agent: |
NeoMutt/20210205-772-2b4c52 |
On Wed, Sep 08, 2021 at 06:54:25AM +0200, Markus Armbruster wrote:
> Intentation.__bool__() is not worth its keep: it has just one user,
Indentation
> which can just as well check .__str__() instead.
>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
> scripts/qapi/common.py | 9 +++------
> 1 file changed, 3 insertions(+), 6 deletions(-)
>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
- [PATCH 0/5] qapi: Another round of minor fixes and cleanups, Markus Armbruster, 2021/09/08
- [PATCH 4/5] tests/qapi-schema: Cover 'not' condition with empty argument, Markus Armbruster, 2021/09/08
- [PATCH 3/5] qapi: Bury some unused code in class Indentation, Markus Armbruster, 2021/09/08
- [PATCH 5/5] qapi: Fix bogus error for 'if': { 'not': '' }, Markus Armbruster, 2021/09/08
- [PATCH 1/5] qapi: Fix a botched type annotation, Markus Armbruster, 2021/09/08
- [PATCH 2/5] qapi: Drop Indentation.__bool__(), Markus Armbruster, 2021/09/08
- Re: [PATCH 2/5] qapi: Drop Indentation.__bool__(),
Eric Blake <=
- Re: [PATCH 0/5] qapi: Another round of minor fixes and cleanups, Marc-André Lureau, 2021/09/08
- Re: [PATCH 0/5] qapi: Another round of minor fixes and cleanups, Markus Armbruster, 2021/09/13