[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 2/6] python/machine: use subprocess.DEVNULL instead of ope
|
From: |
Cleber Rosa |
|
Subject: |
Re: [PATCH v2 2/6] python/machine: use subprocess.DEVNULL instead of open(os.path.devnull) |
|
Date: |
Mon, 17 May 2021 23:33:13 -0400 |
On Mon, May 17, 2021 at 02:48:04PM -0400, John Snow wrote:
> One less file resource to manage, and it helps quiet some pylint >=
> 2.8.0 warnings about not using a with-context manager for the open call.
>
> Signed-off-by: John Snow <jsnow@redhat.com>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> python/qemu/machine.py | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
Reviewed-by: Cleber Rosa <crosa@redhat.com>
signature.asc
Description: PGP signature
- [PATCH v2 0/6] Python: delint python library, John Snow, 2021/05/17
- [PATCH v2 1/6] python/console_socket: avoid one-letter variable, John Snow, 2021/05/17
- [PATCH v2 2/6] python/machine: use subprocess.DEVNULL instead of open(os.path.devnull), John Snow, 2021/05/17
- Re: [PATCH v2 2/6] python/machine: use subprocess.DEVNULL instead of open(os.path.devnull),
Cleber Rosa <=
- [PATCH v2 3/6] python/machine: use subprocess.run instead of subprocess.Popen, John Snow, 2021/05/17
- [PATCH v2 5/6] python/machine: Disable pylint warning for open() in _pre_launch, John Snow, 2021/05/17
- [PATCH v2 6/6] python/machine: disable warning for Popen in _launch(), John Snow, 2021/05/17
- [PATCH v2 4/6] python/console_socket: Add a pylint ignore, John Snow, 2021/05/17
- Re: [PATCH v2 0/6] Python: delint python library, John Snow, 2021/05/18