# HG changeset patch # User Pierre-Yves David # Date 1489615864 25200 # Node ID 2daeab02b4b13b4d4829c216fd0db39e9733ba0b # Parent d4645ae6ba1592a2dc6bc2e82036dd4d090b3a49 hgweb: explicitly tests for None Changeset 7dafa8d0e006 removed the mutable default value, but did not explicitly tested for None. Such implicit testing can introduce semantic and performance issue. We move to an explicit testing for None as recommended by PEP8: https://www.python.org/dev/peps/pep-0008/#programming-recommendations diff -r d4645ae6ba15 -r 2daeab02b4b1 mercurial/hgweb/common.py --- a/mercurial/hgweb/common.py Wed Mar 15 15:10:09 2017 -0700 +++ b/mercurial/hgweb/common.py Wed Mar 15 15:11:04 2017 -0700 @@ -96,7 +96,9 @@ message = _statusmessage(code) Exception.__init__(self, message) self.code = code - self.headers = headers or [] + if headers is None: + headers = [] + self.headers = headers class continuereader(object): def __init__(self, f, write):