emitrevision: add a small closure to check if a base is usable
We will make more use of this and make it more complex too.
--- a/mercurial/utils/storageutil.py Thu Dec 08 15:13:17 2022 +0100
+++ b/mercurial/utils/storageutil.py Mon Nov 28 15:48:51 2022 +0100
@@ -391,6 +391,9 @@
# Set of revs available to delta against.
available = set()
+ def is_usable_base(rev):
+ return rev != nullrev and rev in available
+
for rev in revs:
if rev == nullrev:
continue
@@ -420,7 +423,7 @@
elif deltaparentrev != nullrev:
# Base revision was already emitted in this group. We can
# always safely use the delta.
- if deltaparentrev in available:
+ if is_usable_base(deltaparentrev):
baserev = deltaparentrev
# Base revision is a parent that hasn't been emitted already.