[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 1/2] qemu.py: make 'args' public
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v4 1/2] qemu.py: make 'args' public |
Date: |
Tue, 25 Jul 2017 14:37:08 +0100 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
On Mon, Jul 24, 2017 at 02:44:37PM +0200, Amador Pahim wrote:
> Let's make args public so users can extend it without feeling like
> abusing the internal API.
Nothing is abusing an internal API. PEP8 describes the difference
between public (no underscore), protected aka subclass API (single
underscore), and private fields (single or double underscore):
https://www.python.org/dev/peps/pep-0008/
_args is a protected field. It is perfectly normal for a subclass to
access a protected field in its parent class.
> Signed-off-by: Amador Pahim <address@hidden>
> Reviewed-by: Fam Zheng <address@hidden>
> ---
> scripts/qemu.py | 10 +++++-----
> tests/qemu-iotests/iotests.py | 18 +++++++++---------
> 2 files changed, 14 insertions(+), 14 deletions(-)
Please don't do this, it encourages code duplication. Now arbitrary
users can start accessing the public field directly instead of adding a
reusable interfaces like add_monitor_telnet(), add_fd(), etc.
signature.asc
Description: PGP signature
- [Qemu-devel] [PATCH v4 0/2] scripts/qemu.py fixes and cleanups, Amador Pahim, 2017/07/24
- [Qemu-devel] [PATCH v4 1/2] qemu.py: make 'args' public, Amador Pahim, 2017/07/24
- Re: [Qemu-devel] [PATCH v4 1/2] qemu.py: make 'args' public,
Stefan Hajnoczi <=
- Re: [Qemu-devel] [PATCH v4 1/2] qemu.py: make 'args' public, Amador Pahim, 2017/07/25
- Re: [Qemu-devel] [PATCH v4 1/2] qemu.py: make 'args' public, Cleber Rosa, 2017/07/25
- Re: [Qemu-devel] [PATCH v4 1/2] qemu.py: make 'args' public, Stefan Hajnoczi, 2017/07/26
- Re: [Qemu-devel] [PATCH v4 1/2] qemu.py: make 'args' public, Cleber Rosa, 2017/07/26
- Re: [Qemu-devel] [PATCH v4 1/2] qemu.py: make 'args' public, Amador Pahim, 2017/07/27
- Re: [Qemu-devel] [PATCH v4 1/2] qemu.py: make 'args' public, Amador Pahim, 2017/07/27
[Qemu-devel] [PATCH v4 2/2] qemu.py: cleanup and fixes, Amador Pahim, 2017/07/24