revlog.lookup tweaks

- fast path for binary node ids
- direct lookup for full hex ids
- silly change to bin_id

mercurial/revlog.py

@@ -754,6 +754,15 b' class revlog(object):'
         if isinstance(id, (long, int)):
             # rev
             return self.node(id)
+        if len(id) == 20:
+            # possibly a binary node
+            # odds of a binary node being all hex in ASCII are 1 in 10**25
+            try:
+                node = id
+                r = self.rev(node) # quick search the index
+                return node
+            except RevlogError:
+                pass # may be partial hex id
         try:
             # str(rev)
             rev = int(id)
@@ -764,11 +773,14 b' class revlog(object):'
         except (ValueError, OverflowError):
             pass
         try:
+            if len(id) == 40:
+                # a full hex nodeid?
+                node = bin(id)
+                r = self.rev(node)
+                return node
+            elif len(id) < 40:
             # hex(node)[:...]
-            if len(id) % 2 == 0:
-                bin_id = bin(id)
-            else:
-                bin_id = bin(id[:-1])
+                bin_id = bin(id[:len(id) & ~1]) # grab an even number of digits
             node = None
             for n in self.nodemap:
                 if n.startswith(bin_id) and hex(n).startswith(id):
@@ -780,11 +792,6 b' class revlog(object):'
         except TypeError:
             pass
 
-        # might need fixing if we change hash lengths
-        if len(id) == 20 and id in self.nodemap:
-            # node
-            return id
-
         raise RevlogError(_("No match found"))
 
     def cmp(self, node, text):