D451: revset: remove order information from tree

martinvonz (Martin von Zweigbergk) phabricator at mercurial-scm.org
Tue Aug 22 14:23:27 EDT 2017


martinvonz added a comment.


  I had also wanted to remove the need to pass subset, so I'd be happy so that change.

INLINE COMMENTS

> quark wrote in test-revset.t:2893
> This is caused by `fullreposet` having a default order. If we remove that, it would be optimized to `<baseset [1, 3, 5]>` here.

Does that mean you'll remove the other.sort() in fullreposet.__and__?

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D451

To: quark, #hg-reviewers
Cc: martinvonz, yuja, mercurial-devel


More information about the Mercurial-devel mailing list