# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@fb.com>
# Date 2014-10-07 06:37:39
# Node ID 20d998395ee7fe6912b7a3f25ed571d88ae768bb
# Parent  4f81470e83bf1ef7895be00364e8ef160a6dd79e

revpair: use `first` and `last` instead of direct indexing

This makes it compatible with all smarsets classes.

diff --git a/mercurial/scmutil.py b/mercurial/scmutil.py
--- a/mercurial/scmutil.py
+++ b/mercurial/scmutil.py
@@ -497,9 +497,8 @@ def revpair(repo, revs):
         first = l.max()
         second = l.min()
     else:
-        l = list(l)
-        first = l[0]
-        second = l[-1]
+        first = l.first()
+        second = l.last()
 
     if first is None:
         raise util.Abort(_('empty revision range'))