[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 06/72] python: Add 'fh' to known-good variable names
|
From: |
John Snow |
|
Subject: |
[PATCH v2 06/72] python: Add 'fh' to known-good variable names |
|
Date: |
Tue, 3 Nov 2020 19:34:56 -0500 |
fd and fh are fine: we often use these for "file descriptor" or "file
handle" accordingly. It is rarely the case that you need to enforce a
more semantically meaningful name beyond "This is the file we are using
right now."
While we're here: add comments for all of the non-standard pylint
names. (And the underscore.)
Signed-off-by: John Snow <jsnow@redhat.com>
---
python/setup.cfg | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/python/setup.cfg b/python/setup.cfg
index c8b7215996e9..c963d05527aa 100644
--- a/python/setup.cfg
+++ b/python/setup.cfg
@@ -68,9 +68,10 @@ good-names=i,
k,
ex,
Run,
- _,
- fd,
- c,
+ _, # By convention: Unused variable
+ fh, # fh = open(...)
+ fd, # fd = os.open(...)
+ c, # for c in string: ...
[pylint.similarities]
# Ignore imports when computing similarities.
--
2.26.2
- [PATCH v2 00/72] python: move scripts/qmp to python/qemu/qmp, John Snow, 2020/11/03
- [PATCH v2 03/72] scripts/qmp: redirect qom-xxx scripts to python/qemu/qmp/, John Snow, 2020/11/03
- [PATCH v2 02/72] python/qmp: add qom script entry points, John Snow, 2020/11/03
- [PATCH v2 04/72] scripts/qom-fuse: apply isort rules, John Snow, 2020/11/03
- [PATCH v2 05/72] scripts/qom-fuse: apply flake8 rules, John Snow, 2020/11/03
- [PATCH v2 01/72] python/qmp: Add qom script rewrites, John Snow, 2020/11/03
- [PATCH v2 08/72] scripts/qom-fuse: Add docstrings, John Snow, 2020/11/03
- [PATCH v2 06/72] python: Add 'fh' to known-good variable names,
John Snow <=
- [PATCH v2 07/72] scripts/qom-fuse: Apply pylint rules, John Snow, 2020/11/03
- [PATCH v2 09/72] scripts/qom-fuse: Convert to QOMCommand, John Snow, 2020/11/03
- [PATCH v2 12/72] scripts/qom-fuse: add static type hints, John Snow, 2020/11/03
- [PATCH v2 11/72] scripts/qom-fuse: ensure QOMFuse.read always returns bytes, John Snow, 2020/11/03
- [PATCH v2 16/72] python: add optional fuse dependency, John Snow, 2020/11/03
- [PATCH v2 10/72] scripts/qom-fuse: use QOMCommand.qom_list(), John Snow, 2020/11/03
- [PATCH v2 13/72] scripts/qom-fuse: move to python/qemu/qmp/qom_fuse.py, John Snow, 2020/11/03
- [PATCH v2 15/72] python: add fuse command to 'qom' tools, John Snow, 2020/11/03
- [PATCH v2 18/72] scripts/qemu-ga-client: apply (most) flake8 rules, John Snow, 2020/11/03
- [PATCH v2 20/72] scripts/qemu-ga-client: replace deprecated optparse with argparse, John Snow, 2020/11/03