stack: remove unnecessary reverse() predicate

Stack already sorts revisions, so no need to do it twice.

This change was a part of D2400, which didn't land for other reasons. See also
D2399, where this change was suggested.

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

mercurial/stack.py

@@ -1,29 +1,29 b''
 # stack.py - Mercurial functions for stack definition
 #
 #  Copyright Matt Mackall <[e-mail]> and other
 #
 # This software may be used and distributed according to the terms of the
 # GNU General Public License version 2 or any later version.
 
 from __future__ import absolute_import
 
 from . import (
     revsetlang,
     scmutil,
 )
 
 def getstack(repo, rev=None):
     """return a sorted smartrev of the stack containing either rev if it is
     not None or the current working directory parent.
 
     The stack will always contain all drafts changesets which are ancestors to
     the revision and are not merges.
     """
     if rev is None:
         rev = '.'
 
-    revspec = 'reverse(only(%s) and not public() and not ::merge())'
+    revspec = 'only(%s) and not public() and not ::merge()'
     revset = revsetlang.formatspec(revspec, rev)
     revisions = scmutil.revrange(repo, [revset])
     revisions.sort()
     return revisions