D1678: revset: pass pre-optimized tree in revset.matchany()
pulkit (Pulkit Goyal)
phabricator at mercurial-scm.org
Tue Dec 19 07:15:45 EST 2017
pulkit abandoned this revision.
pulkit added a comment.
In https://phab.mercurial-scm.org/D1678#29063, @yuja wrote:
> > Since we are calling them at each command level, it will be better to call unhidehashlikerevs() in `scmutil.revrange|revsingle`.
> I thought about that, but `scmutil.rev*()` would have to return new filtered `repo`
> object, which didn't seem nice. And mutating a given `repo` argument has the
> original problem, when to discard unhidden revisions. So my conclusion was a
> separate function.
> > We will still need to store the filtername in dispatch and then later on
> > decide on basis of that whether we want to uhide things.
> Doesn't each command know whether it should unhide or not?
Thanks for all the ideas, I have send the new series as https://phab.mercurial-scm.org/D1730 - https://phab.mercurial-scm.org/D1735 as they are entirely different changesets.
To: pulkit, #hg-reviewers
Cc: yuja, mercurial-devel
More information about the Mercurial-devel