<div dir="ltr"><br><br><div class="gmail_quote">On Sat, May 16, 2015 at 2:34 PM Adrian Buehlmann <<a href="mailto:adrian@cadifra.com">adrian@cadifra.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 2015-05-16 23:08, Martin von Zweigbergk wrote:<br>
><br>
><br>
> On Sat, May 16, 2015 at 11:38 AM Augie Fackler <<a href="mailto:raf@durin42.com" target="_blank">raf@durin42.com</a><br>
> <mailto:<a href="mailto:raf@durin42.com" target="_blank">raf@durin42.com</a>>> wrote:<br>
><br>
><br>
>     On May 16, 2015, at 2:33 PM, Martin von Zweigbergk<br>
>     <<a href="mailto:martinvonz@google.com" target="_blank">martinvonz@google.com</a> <mailto:<a href="mailto:martinvonz@google.com" target="_blank">martinvonz@google.com</a>>> wrote:<br>
><br>
>     > I've pushed this to the clowncopter, thanks. Will send my<br>
>     follow-up patch once this patch is in Matt's repo.<br>
><br>
>     If you want, I’d be happy to review the removal of the name forward<br>
>     series and then you could clowncopterize it.<br>
><br>
><br>
> Yes, please. Take a look at<br>
> <a href="https://bitbucket.org/martinvonz/hg/commits/0ca8410ea3455bcef5b9335ca6d9aa2de4adc2c9" target="_blank">https://bitbucket.org/martinvonz/hg/commits/0ca8410ea3455bcef5b9335ca6d9aa2de4adc2c9</a>.<br>
> Thanks!<br>
<br>
Looks good enough to me. If the testsuite passes, just push that thing.<br></blockquote><div><br></div><div>Pushed to the clowncopter. Thanks for reviewing.</div><div><br></div><div>What is it that makes it only "good enough"? Is the imports you are not quite happy with?</div></div></div>