<div dir="ltr">That's an interesting article and also meaningful for coders. If I have a patch more than 200 or 300 lines, to split this may be a good idea.<div><br></div><div>Some time, an easy patch with a little more lines would prevent more reviewers to think about it.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 15, 2013 at 10:12 AM, Robert Collins <span dir="ltr"><<a href="mailto:robertc@robertcollins.net" target="_blank">robertc@robertcollins.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>This may interest data-driven types here.<br><br><a href="https://www.ibm.com/developerworks/rational/library/11-proven-practices-for-peer-review/" target="_blank">https://www.ibm.com/developerworks/rational/library/11-proven-practices-for-peer-review/</a><br>

<br></div>Note specifically the citation of 200-400 lines as the knee of the review effectiveness curve: that's lower than I thought - I thought 200 was clearly fine - but no.<br><br></div>-Rob<span class="HOEnZb"><font color="#888888"><br clear="all">
<div><div>
<br>-- <br>Robert Collins <<a href="mailto:rbtcollins@hp.com" target="_blank">rbtcollins@hp.com</a>><br>Distinguished Technologist<br>HP Converged Cloud
</div></div></font></span></div>
<br>_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><font color="#444444">Gareth</font><div><font color="#444444"><br></font><div><i><font color="#444444">Cloud Computing, OpenStack, Fitness, Basketball</font></i></div>
<div><i><font color="#666666">OpenStack contributor</font></i></div><div><i><font color="#666666">Company: <a href="http://www.ustack.com" target="_blank">UnitedStack</a></font></i></div><div><i><font color="#999999">My promise: if you find any spelling or grammar mistakes in my email from Mar 1 2013, notify me </font></i></div>
<div><i><font color="#999999">and I'll donate $1 or ￥1 to an open organization you specify.</font></i></div></div></div>
</div>