Skip to content

Commit

Permalink
python/qmp: remove pylint workaround from legacy.py
Browse files Browse the repository at this point in the history
Pylint upgraded recently (2.13.z) and having a pylint: disable comment
in the middle of an argument field causes it some grief (It appears to
stop parsing when it encounters it, causing some syntax problems). Since
the duplicate line threshold was bumped up in 22305c2, we don't
need this workaround anymore. Drop it.

Signed-off-by: John Snow <jsnow@redhat.com>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@openvz.org>
Message-id: 20220330172812.3427355-10-jsnow@redhat.com
Signed-off-by: John Snow <jsnow@redhat.com>
  • Loading branch information
jnsnow committed Apr 21, 2022
1 parent b1a9b1f commit 4743077
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions python/qemu/qmp/legacy.py
Expand Up @@ -106,8 +106,6 @@ def __enter__(self: _T) -> _T:
return self

def __exit__(self,
# pylint: disable=duplicate-code
# see https://github.com/PyCQA/pylint/issues/3619
exc_type: Optional[Type[BaseException]],
exc_val: Optional[BaseException],
exc_tb: Optional[TracebackType]) -> None:
Expand Down

0 comments on commit 4743077

Please sign in to comment.