<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Aug 9, 2014 at 4:33 AM, Deepak Panickal <span dir="ltr"><<a href="mailto:deepak@codeplay.com" target="_blank">deepak@codeplay.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Yeah, I understand. <div><br></div><div>The lldb-mi driver is a sub project which only we have been committing to and working on primarily. It’s separate from the core files of LLDB. Which is why I committed a bigger patch.</div>
<div><br></div><div><div>I agree with you that smaller incremental patches are always better, and would try to stick to that as much as possible.</div></div></div></blockquote></div><br>I understand that is the historical reason why, however I think that "as much as possible" should be "always".</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">If you think you have a patch which cannot be split up, I would send it for pre-commit review and ask the reviewers if they see any way to split it up, and if not, are OK with it being landed as a large patch. It is extremely rare that this actually can't be done.</div>
</div>