# HG changeset patch # User Augie Fackler # Date 1573756077 18000 # Node ID 23ad4f0c1578b2c1048ade3629291cd819e83784 # Parent a9b14ef701d12bdb5a8f4e4582c3e98fdd425ed2 debugcommands: add assertions to convince pytype peer is not None This function is moderately annoyingly defined, and peer is set up iff we're not in raw-proto mode. That's fine, but it confuses pytype. Adding these assertions is a low-overhead way to convince pytype we're doing reasonable things. Differential Revision: https://phab.mercurial-scm.org/D7407 diff -r a9b14ef701d1 -r 23ad4f0c1578 mercurial/debugcommands.py --- a/mercurial/debugcommands.py Wed Nov 13 22:35:15 2019 -0500 +++ b/mercurial/debugcommands.py Thu Nov 14 13:27:57 2019 -0500 @@ -4161,6 +4161,7 @@ _(b'sending batch with %d sub-commands\n') % len(batchedcommands) ) + assert peer is not None for i, chunk in enumerate(peer._submitbatch(batchedcommands)): ui.status( _(b'response #%d: %s\n') % (i, stringutil.escapestr(chunk)) @@ -4241,6 +4242,7 @@ ) elif action == b'close': + assert peer is not None peer.close() elif action == b'readavailable': if not stdout or not stderr: