<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 29, 2013 at 12:57 PM, Stephen Kelly <span dir="ltr"><<a href="mailto:steveire@gmail.com" target="_blank">steveire@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">Nicolas Desprès wrote:<br>
> It was fastest because it was not doing the right thing. I tried to patch<br>
> it properly and the benchmark are the same whether we use the default<br>
> comparison functor or mine.<br>
><br>
> So I think you can merge it like that. I have pushed a new version without<br>
> the comment.<br>
><br>
<br>
</div>I still haven't tried it, but there are still style issues. </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im HOEnZb"><br>
* Don't put an else after a return<br>
* Wrap single line blocks in {}<br></div></blockquote><div><br></div><div style>Fixed and force-pushed. Sorry for the inconvenience. I am not used to this style yet.</div><div style><br></div><div style>Thanks for the review.</div>

<div style>-Nico</div><div style><br></div></div>
</div></div>