# HG changeset patch
# User Mads Kiilerich <madski@unity3d.com>
# Date 1358365301 -3600
# Node ID 95b8629fd2deda0c089895730e33f8b3ba03caf6
# Parent  c448562ad85b60a26eb2c814952e6facef2e9c03
bundlerepo: use rev instead of node for iteration in revision()

Avoids some lookups and avoids using rev while iterating.

diff -r c448562ad85b -r 95b8629fd2de mercurial/bundlerepo.py
--- a/mercurial/bundlerepo.py	Wed Jan 16 20:41:41 2013 +0100
+++ b/mercurial/bundlerepo.py	Wed Jan 16 20:41:41 2013 +0100
@@ -109,17 +109,16 @@
 
         text = None
         chain = []
-        iter_node = node
+        iterrev = rev
         # reconstruct the revision if it is from a changegroup
-        while rev in self.basemap:
-            if self._cache and self._cache[0] == iter_node:
+        while iterrev in self.basemap:
+            if self._cache and self._cache[1] == iterrev:
                 text = self._cache[2]
                 break
-            chain.append(rev)
-            iter_node = self.basemap[rev]
-            rev = self.rev(iter_node)
+            chain.append(iterrev)
+            iterrev = self.rev(self.basemap[iterrev])
         if text is None:
-            text = revlog.revlog.revision(self, iter_node)
+            text = revlog.revlog.revision(self, iterrev)
 
         while chain:
             delta = self._chunk(chain.pop())
@@ -128,9 +127,9 @@
         p1, p2 = self.parents(node)
         if node != revlog.hash(text, p1, p2):
             raise error.RevlogError(_("integrity check failed on %s:%d")
-                                     % (self.datafile, self.rev(node)))
+                                     % (self.datafile, rev))
 
-        self._cache = (node, self.rev(node), text)
+        self._cache = (node, rev, text)
         return text
 
     def addrevision(self, text, transaction, link, p1=None, p2=None, d=None):