[PATCH 6 of 8] hidden: remove _consistencyblockers()
Jun Wu
quark at fb.com
Tue May 30 13:47:50 EDT 2017
Excerpts from Pierre-Yves David's message of 2017-05-29 19:16:06 +0200:
> Ho, this is not a set‽ (that is what I've been missing).
>
> Hum, we should probably stick to pure python set for the return of
> phasecase here and only upgrade them to smartset in revset.
smartset is better because:
- smartset is ordered. If we return a set, some code paths still sort
them, which is a waste of time.
- smartset could be lazy and therefore more flexible.
> The hidding/obsolete logic are not using these set lazily so having a
> smart is likely to just add some overhead.
>
> Cheers
>
More information about the Mercurial-devel
mailing list