From: John Snow <js...@redhat.com> In truth, if you don't do this, you'll just get a TypeError exception. Now, you'll get an AssertionError.
Is this tangibly better? No. Does mypy complain less? Yes. Signed-off-by: John Snow <js...@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com> Message-Id: <20200514055403.18902-21-js...@redhat.com> Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com> --- python/qemu/qmp.py | 1 + 1 file changed, 1 insertion(+) diff --git a/python/qemu/qmp.py b/python/qemu/qmp.py index a634c4e26c..e64b6b5faa 100644 --- a/python/qemu/qmp.py +++ b/python/qemu/qmp.py @@ -94,6 +94,7 @@ def __negotiate_capabilities(self): raise QMPCapabilitiesError def __json_read(self, only_event=False): + assert self.__sockfile is not None while True: data = self.__sockfile.readline() if not data: -- 2.21.3