# Patchwork [1,of,9,cah] ancestors: extract candidates function as commonancestorsheads

Submitter Mads Kiilerich April 17, 2014, 6:07 p.m. <64911a12dc2849b05383.1397758073@localhost.localdomain> mbox | patch /patch/4395/ Accepted 64911a12dc2849b053830a30ca3d19377fa12a1b show

Mads Kiilerich - April 17, 2014, 6:07 p.m.
```# HG changeset patch
# Date 1397756996 -7200
#      Thu Apr 17 19:49:56 2014 +0200
# Node ID 64911a12dc2849b053830a30ca3d19377fa12a1b
# Parent  098a274764b3813ca0788728d88187d21b9785e5
ancestors: extract candidates function as commonancestorsheads
```

## Patch

```diff --git a/mercurial/ancestor.py b/mercurial/ancestor.py
--- a/mercurial/ancestor.py
+++ b/mercurial/ancestor.py
@@ -9,6 +9,62 @@  import heapq
import util
from node import nullrev

+    """Returns a set with the heads of all common ancestors of all nodes,
+    heads(::nodes and ::nodes and ...) .
+
+    pfunc must return a list of parent vertices for a given vertex.
+    """
+    if not isinstance(nodes, set):
+        nodes = set(nodes)
+    if nullrev in nodes:
+        return set()
+    if len(nodes) <= 1:
+        return nodes
+
+    allseen = (1 << len(nodes)) - 1
+    seen =  * (max(nodes) + 1)
+    for i, n in enumerate(nodes):
+        seen[n] = 1 << i
+    poison = 1 << (i + 1)
+
+    gca = set()
+    interesting = len(nodes)
+    nv = len(seen) - 1
+    while nv >= 0 and interesting:
+        v = nv
+        nv -= 1
+        if not seen[v]:
+            continue
+        sv = seen[v]
+        if sv < poison:
+            interesting -= 1
+            if sv == allseen:
+                sv |= poison
+                if v in nodes:
+                    # history is linear
+                    return set([v])
+        if sv < poison:
+            for p in pfunc(v):
+                sp = seen[p]
+                if p == nullrev:
+                    continue
+                if sp == 0:
+                    seen[p] = sv
+                    interesting += 1
+                elif sp != sv:
+                    seen[p] |= sv
+        else:
+            for p in pfunc(v):
+                if p == nullrev:
+                    continue
+                sp = seen[p]
+                if sp and sp < poison:
+                    interesting -= 1
+                seen[p] = sv
+    return gca
+
def ancestors(pfunc, *orignodes):
"""
Returns the common ancestors of a and b that are furthest from a
@@ -16,57 +72,6 @@  def ancestors(pfunc, *orignodes):

pfunc must return a list of parent vertices for a given vertex.
"""
-    if not isinstance(orignodes, set):
-        orignodes = set(orignodes)
-    if nullrev in orignodes:
-        return set()
-    if len(orignodes) <= 1:
-        return orignodes
-
-    def candidates(nodes):
-        allseen = (1 << len(nodes)) - 1
-        seen =  * (max(nodes) + 1)
-        for i, n in enumerate(nodes):
-            seen[n] = 1 << i
-        poison = 1 << (i + 1)
-
-        gca = set()
-        interesting = len(nodes)
-        nv = len(seen) - 1
-        while nv >= 0 and interesting:
-            v = nv
-            nv -= 1
-            if not seen[v]:
-                continue
-            sv = seen[v]
-            if sv < poison:
-                interesting -= 1
-                if sv == allseen:
-                    sv |= poison
-                    if v in nodes:
-                        # history is linear
-                        return set([v])
-            if sv < poison:
-                for p in pfunc(v):
-                    sp = seen[p]
-                    if p == nullrev:
-                        continue
-                    if sp == 0:
-                        seen[p] = sv
-                        interesting += 1
-                    elif sp != sv:
-                        seen[p] |= sv
-            else:
-                for p in pfunc(v):
-                    if p == nullrev:
-                        continue
-                    sp = seen[p]
-                    if sp and sp < poison:
-                        interesting -= 1
-                    seen[p] = sv
-        return gca
-
def deepest(nodes):
interesting = {}
count = max(nodes) + 1
@@ -123,7 +128,7 @@  def ancestors(pfunc, *orignodes):
k |= i
return set(n for (i, n) in mapping if k & i)

-    gca = candidates(orignodes)