<div dir="ltr">Thanks for the review!<div>I have committed revision 214086.</div><div><br></div><div>I had thought about committing anyway and having a post-commit review, but I still don't have a long track record so I thought it was safer to ask for a pre-commit review.</div>
<div><br></div><div>Cheers,</div><div>    Dario Domizioli</div><div>    SN Systems - Sony Computer Entertainment Group</div><div><br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On 28 July 2014 15:22, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.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="">On Mon, Jul 28, 2014 at 10:16 AM, Dario Domizioli<br>
<<a href="mailto:dario.domizioli@gmail.com">dario.domizioli@gmail.com</a>> wrote:<br>
><br>
> Ping. :-)<br>
><br>
> It's just a test improvement patch so it shouldn't be controversial.<br>
<br>
</div>LGTM!<br>
<br>
Btw, improved test coverage is something that can usually be dealt<br>
with post-commit, especially when it's functionality you've been<br>
involved with before.<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div><br></div>