<div dir="ltr">It was only 25 :)<div><br></div><div>Sorry, I'll try to limit the commit frequency in the future.  For reference, what would you recommend for spacing out larger collections of patches?  15 minutes per 3?  30 minutes per 5?</div>
<div><br></div><div>Re: the tests, the commits that do not contain new or modified tests should be clean-ups or structural changes where correctness is checked by existing tests.  Thanks for looking through them!</div></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 27, 2014 at 3:17 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</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">This makes it *really* hard to do code review etc. They arrive on the email list out-of-order, and the build bots end up picking up huge chunks rather than having incremental coverage.<div>
<br></div><div>I'm having a hard time getting through all of these, but a bunch of the commits you just pushed seem to not have tests associated with them as well. I will try to cite the ones missing tests, but again, this becomes *really* hard when you do a drop of so many commits.</div>

</div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><br><div>Thanks,</div><div><br></div><div>Justin Holewinski</div>
</div>