python/qmp: remove pylint workaround from legacy.py
authorJohn Snow <jsnow@redhat.com>
Wed, 30 Mar 2022 17:28:12 +0000 (13:28 -0400)
committerJohn Snow <jsnow@redhat.com>
Thu, 21 Apr 2022 15:01:00 +0000 (11:01 -0400)
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 22305c2a081b, 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>
python/qemu/qmp/legacy.py

index a8629b44dffcf80ee1670be651c66b7477b7e70e..03b5574618fad3fec50ccf59d253a92119be6167 100644 (file)
@@ -106,8 +106,6 @@ class QEMUMonitorProtocol:
         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: