<div dir="ltr"><br><br><div class="gmail_quote">On Wed, May 13, 2015 at 9:21 PM Mike Hommey <<a href="mailto:mh@glandium.org">mh@glandium.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, May 12, 2015 at 01:10:25PM -0700, Pierre-Yves David wrote:<br>
> >Because I wasn't sure the approach was right, and sure enough, 2<br>
> >messages later I figured it introduced a regression. And maybe we'd be<br>
> >better off with a more aggressive approach, trying to minimize the slow<br>
> >path even more.<br>
> ><br>
> >Mike<br>
> ><br>
> >PS: the funny thing is that the slow path is based on data that may not<br>
> >be accurate (as the mozilla-central history demonstrates), but<br>
> >fortunately, /shouldn't/ indicate new files that aren't new. That's<br>
> >funny because if it was happening, then the slow path would need to be<br>
> >even slower.<br>
><br>
> I think martin did some work in this area recently. And your patch does not<br>
> apply anymore.<br>
><br>
> Can you have a look at what martin did and wrote about it and come back with<br>
> a V2 and new data?<br>
<br>
AIUI, the changes he did made the slow path be used in more cases than<br>
before,</blockquote><div><br></div><div>I'm currently away from work with limited time to check, but I think I only refactored and documented the behavior. I don't remember intentionally changing it.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> so there is no change wrt the original issue. Running the same<br>
commands as in the original patch message still lead to the same<br>
timings. OTOH the original patch was also buggy, so /that/ would need to<br>
be addressed. At this point, I'd rather someone who actually know this<br>
code took a look.<br></blockquote><div><br></div><div>I'll set a reminder for myself to look into this some time after I get back to work.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Mike<br>
_______________________________________________<br>
Mercurial-devel mailing list<br>
<a href="mailto:Mercurial-devel@selenic.com" target="_blank">Mercurial-devel@selenic.com</a><br>
<a href="http://selenic.com/mailman/listinfo/mercurial-devel" target="_blank">http://selenic.com/mailman/listinfo/mercurial-devel</a><br>
</blockquote></div></div>