<div dir="ltr"><br><br><div class="gmail_quote">On Fri, Feb 27, 2015 at 3:57 PM Matt Mackall <<a href="mailto:mpm@selenic.com">mpm@selenic.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, 2015-02-26 at 16:02 -0800, Martin von Zweigbergk wrote:<br>
> # HG changeset patch<br>
> # User Martin von Zweigbergk <<a href="mailto:martinvonz@google.com" target="_blank">martinvonz@google.com</a>><br>
> # Date 1424802971 28800<br>
> #      Tue Feb 24 10:36:11 2015 -0800<br>
> # Node ID 1d3fe60a20d4f6284bc295af27f9f2<u></u>72af30dcdd<br>
> # Parent  ef7dde10dc5941491a19dbd17ccb47<u></u>a776f0dadd<br>
> manifest: postpone calling array.array() until needed<br>
><br>
> We currently convert a string to an array in manifest.read(), only to<br>
> store the result in the manifest cache. In most cases, the array will<br>
> not be used, so let's store the original string instead. Storage cost<br>
> should be similar assuming strings are stored in memory as byte<br>
> arrays. The only disadvantage should be that the conversion can be<br>
> done several times if multiple new revisions are created based on the<br>
> same cached revision.<br>
<br>
I've dropped this one because it looks like it de-optimized back-to-back<br>
commits, for instance of the sort done by pushing a stack of mq patches.<br></blockquote><div><br></div><div>I see the problem now. Sorry, I should have caught that.</div><div><br></div><div>It was a tangent to my other work, so I won't attempt a V2.</div></div></div>