comparison mercurial/manifest.py @ 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 888bd39ed555
children c86256bd4eb8
comparison
equal deleted inserted replaced
44241:aea79f41ee55 44257:dbbae122f5e4
458 return len(self._lm) != 0 458 return len(self._lm) != 0
459 459
460 __bool__ = __nonzero__ 460 __bool__ = __nonzero__
461 461
462 def __setitem__(self, key, node): 462 def __setitem__(self, key, node):
463 self._lm[key] = node, self.flags(key, b'') 463 self._lm[key] = node, self.flags(key)
464 464
465 def __contains__(self, key): 465 def __contains__(self, key):
466 if key is None: 466 if key is None:
467 return False 467 return False
468 return key in self._lm 468 return key in self._lm
593 try: 593 try:
594 return self._lm[key][0] 594 return self._lm[key][0]
595 except KeyError: 595 except KeyError:
596 return default 596 return default
597 597
598 def flags(self, key, default=b''): 598 def flags(self, key):
599 try: 599 try:
600 return self._lm[key][1] 600 return self._lm[key][1]
601 except KeyError: 601 except KeyError:
602 return default 602 return b''
603 603
604 def copy(self): 604 def copy(self):
605 c = manifestdict() 605 c = manifestdict()
606 c._lm = self._lm.copy() 606 c._lm = self._lm.copy()
607 return c 607 return c