[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mercurial-devel
Subject:    Re: [PATCH 1 of 6] dirstate: rename dirs to iterdirs, and return only dir names
From:       "Bryan O'Sullivan" <bos () serpentine ! com>
Date:       2013-03-29 23:09:41
Message-ID: CACw0niLvKFJskqGKjHpwh6LPLMNzrc+Vc3+rHHTB84GrxAwjVA () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


On Thu, Mar 28, 2013 at 6:22 PM, Bryan O'Sullivan <bos@serpentine.com>wrote:

> The refcounts that we maintain behind the scenes are not public, so make
> sure we don't expose them to callers (who don't care anyway).
>

After some further code inspection (thanks to Siddharth), it turns out that
context.changectx has an almost identical need to compute a bunch of dirs.

To accommodate that cleanly, I'm going to rework this patchset a little.

[Attachment #5 (text/html)]

On Thu, Mar 28, 2013 at 6:22 PM, Bryan O&#39;Sullivan <span dir="ltr">&lt;<a \
href="mailto:bos@serpentine.com" target="_blank">bos@serpentine.com</a>&gt;</span> \
wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 \
0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <div id=":zm">The refcounts that \
we maintain behind the scenes are not public, so make<br> sure we don&#39;t expose \
them to callers (who don&#39;t care anyway).<br></div></blockquote></div><br>After \
some further code inspection (thanks to Siddharth), it turns out that \
context.changectx has an almost identical need to compute a bunch of dirs.<div> \
<br></div><div>To accommodate that cleanly, I&#39;m going to rework this patchset a \
little.</div>



_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@selenic.com
http://selenic.com/mailman/listinfo/mercurial-devel


[prev in list] [next in list] [prev in thread] [next in thread]