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 <apa...@redhat.com> > Reviewed-by: Fam Zheng <f...@redhat.com> > --- > 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
