<div dir="ltr">On 14 February 2014 19:34, Gonzalo Odiard <span dir="ltr"><<a href="mailto:gonzalo@laptop.org" target="_blank">gonzalo@laptop.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div><div><div><br></div>* Backup<br><br>
</div><div>This was posted a while ago but no one reviewed yet. Tests are missing. Gonzalo, is it supposed to be otherwise complete? I feel we are risking to miss the train here. Any reviewer volunteering to look at it? I'm probably not going to be able to do that soon. <br>


</div><div><br></div></div></div></blockquote><div><br></div></div><div>I have not added test because never had a review.<br></div><div>I understand is a big piece of code, but the pull request is more than 2 months old :(</div>

<div><br></div></div></div></div></blockquote><div><br></div><div>Yes that's bad. But how many patches did you review in the last 2 months? :)<br><br></div><div>Reviewing patches have to become a much higher priority for everyone or the community will never grow and patches will keep being reviewed slowly. I'm not picking on you, but that's the way it is.<br>
</div></div></div></div>