<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, Jun 30, 2016 at 4:18 PM Justin Bogner <<a href="mailto:mail@justinbogner.com">mail@justinbogner.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I find this new API very confusing, and the addition you've made to the<br>
comment doesn't even mention what happens with Chain.<br>
<br>
Wouldn't it make more sense to update Chain outside of<br>
findBetterNeighbourChains anyway?</blockquote><div><br></div><div>Done. Part of the reason I included the Chain into <span style="line-height:1.5">findBetterNeighbourChains</span><span style="line-height:1.5"> is that the caller doesn't have to remember to update the chain (similar to SelectionDAG::getStore, etc). But I don't have strong opinion on this.</span></div></div></div>