# HG changeset patch # User Gábor Stefanik <gabor.stefa...@nng.com> # Date 1518443649 -3600 # Mon Feb 12 14:54:09 2018 +0100 # Node ID fe38aeeb1586464769caa6e9bb819078028fc858 # Parent 1085c904d8c04d51c6897027fe9c7bae0964b64b [b] ignore close() on non-open clients (issue5751)
Closing a client twice currently triggers a rather confusing exception. Instead, follow the convention set by Python's file objects, and ignore close() commands on non-open clients. diff -r 1085c904d8c0 -r fe38aeeb1586 hglib/client.py --- a/hglib/client.py Thu Feb 01 15:10:02 2018 -0500 +++ b/hglib/client.py Mon Feb 12 14:54:09 2018 +0100 @@ -294,6 +294,8 @@ return self._close()[0] def _close(self): + if not self.server: + return 0, '' _sout, serr = self.server.communicate() ret = self.server.returncode self.server = None ________________________________ This message, including its attachments, is confidential and the property of NNG Llc. For more information please read NNG's email policy here: https://www.nng.com/email-policy/ By responding to this email you accept the email policy. _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel