[PATCH 2 of 2] manifest: drop the now-unused find method

Bryan O'Sullivan bos at serpentine.com
Tue Mar 12 18:51:00 CDT 2013


On Tue, Mar 12, 2013 at 3:37 PM, Durham Goode <durham at fb.com> wrote:

> Looks good.  I don't see anywhere else in the code that uses it.
>

It turns out to affect grep performance after all, at least in the
small-number-of-changes case. That's ... awkward.

So we need something like manifest.find to help (a) the common case of just
a few changes to the manifest, but we also need something for (b) large
numbers of changes, but it has to not clobber performance for case (a).
This needs some more thinking :-(
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20130312/6b93dfe5/attachment.html>


More information about the Mercurial-devel mailing list