On Tue, Aug 28, 2012 at 12:20 PM, Leah Xue <span dir="ltr"><<a href="mailto:leahxue@fb.com" target="_blank">leahxue@fb.com</a>></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=":5f">+        files = foldchanges(ui, repo, oldctx.parents()[0].node(), ha, opts)<br></div></blockquote></div><br><div>This patch mostly looks great, but here's something I just noticed: you can replace oldctx.parents()[0].node() with oldctx.p1().node() for a slight improvement in clarity.</div>