# HG changeset patch
# User Martin von Zweigbergk <martinvonz@google.com>
# Date 1532110653 25200
# Node ID 83a505b5cf85ab172e30dedf4d940070f027f1eb
# Parent  65ed2fcb9032ae3822fbaf5aac50d8feb0f35cbf
revlog: don't include trailing nullrev in revlog.revs(stop=len(revlog))

This was an odd side effect of the nullid entry that's in the
index. The existing callers (mostly repair.py) seem to have handled it
fine. It doesn't seem intentional, and it's pretty surprising, so
let's remove that surprise.

Differential Revision: https://phab.mercurial-scm.org/D4015

diff -r 65ed2fcb9032 -r 83a505b5cf85 mercurial/revlog.py
--- a/mercurial/revlog.py	Mon Jul 16 14:15:29 2018 -0400
+++ b/mercurial/revlog.py	Fri Jul 20 11:17:33 2018 -0700
@@ -1070,12 +1070,15 @@
     def revs(self, start=0, stop=None):
         """iterate over all rev in this revlog (from start to stop)"""
         step = 1
+        length = len(self)
         if stop is not None:
             if start > stop:
                 step = -1
             stop += step
+            if stop > length:
+                stop = length
         else:
-            stop = len(self)
+            stop = length
         return xrange(start, stop, step)
 
     @util.propertycache