Mercurial > hg
changeset 44257:dbbae122f5e4
manifest: remove optional default= argument on flags(path)
It had only one caller inside manifest.py, and treemanifest was
actually incorrectly implemented. treemanifest is still missing the
fastdelta() method from the interface (and so doesn't yet conform),
but this is at least progress.
Differential Revision: https://phab.mercurial-scm.org/D8069
| author | Augie Fackler <augie@google.com> |
|---|---|
| date | Mon, 03 Feb 2020 22:16:36 -0500 |
| parents | aea79f41ee55 |
| children | 9659ec161644 |
| files | mercurial/interfaces/repository.py mercurial/manifest.py |
| diffstat | 2 files changed, 5 insertions(+), 5 deletions(-) [+] |
line wrap: on
line diff
--- a/mercurial/interfaces/repository.py Mon Feb 03 11:56:02 2020 -0500 +++ b/mercurial/interfaces/repository.py Mon Feb 03 22:16:36 2020 -0500 @@ -1027,8 +1027,8 @@ def get(path, default=None): """Obtain the node value for a path or a default value if missing.""" - def flags(path, default=b''): - """Return the flags value for a path or a default value if missing.""" + def flags(path): + """Return the flags value for a path (default: empty bytestring).""" def copy(): """Return a copy of this manifest."""
--- a/mercurial/manifest.py Mon Feb 03 11:56:02 2020 -0500 +++ b/mercurial/manifest.py Mon Feb 03 22:16:36 2020 -0500 @@ -460,7 +460,7 @@ __bool__ = __nonzero__ def __setitem__(self, key, node): - self._lm[key] = node, self.flags(key, b'') + self._lm[key] = node, self.flags(key) def __contains__(self, key): if key is None: @@ -595,11 +595,11 @@ except KeyError: return default - def flags(self, key, default=b''): + def flags(self, key): try: return self._lm[key][1] except KeyError: - return default + return b'' def copy(self): c = manifestdict()