<div dir="ltr">Yes. Do a separate review for each problem, please.<div><br></div><div>Thanks,</div><div>Ilia</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 10, 2015 at 3:38 AM,  <span dir="ltr"><<a href="mailto:jingham@apple.com" target="_blank">jingham@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Note, this also looks like a pretty big patch that covers a bunch of problems, which is going to make it pretty hard to review.  It would be best to submit separate patches for each issue you addressed, saying what the issue was.  If you needed to do some more general infrastructure work, separate that out and submit that as well.<br>
<br>
Thanks for working on this, BTW!<br>
<span class="HOEnZb"><font color="#888888"><br>
Jim<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
> On Mar 9, 2015, at 5:34 PM, Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br>
><br>
> Hi Chuck, I'll update the instructions for submitting a patch.  In the meantime, the best way to get the ball rolling is to:<br>
><br>
> a) Create a Phabricator account<br>
> b) Subscribe to lldb-commits mailing list<br>
> c) Upload your patch to Phabricator (click Differential on the left hand pane, and then click "+ Create Diff" in the top right corner, and attach your patch there).<br>
> d) When you get to the step where you fill in reviewers, etc please make sure lldb-commits is CC'ed, and include some people as reviewers.  For this particular change, I would suggest adding Jim Ingham for the line in BreakpointLocation.cpp and abidh for the MI changes  (we should probably make a CODE_OWNERS.txt in root lldb so you'll be able to figure this out for yourself).   People will add additional reviewers as necessary.<br>
><br>
> On Mon, Mar 9, 2015 at 5:21 PM Chuck Ries <<a href="mailto:Chuck.Ries@microsoft.com">Chuck.Ries@microsoft.com</a>> wrote:<br>
> Hello all,<br>
><br>
> The instructions at <a href="http://lldb.llvm.org/source.html" target="_blank">http://lldb.llvm.org/source.html</a> are very minimal on how to contribute to lldb, and after reading the instructions at <a href="http://llvm.org/docs/DeveloperPolicy.html#making-and-submitting-a-patch" target="_blank">http://llvm.org/docs/DeveloperPolicy.html#making-and-submitting-a-patch</a>, I am unsure what applies and what does not.<br>
><br>
><br>
><br>
> At this point I have generated and attached a patch. The changes are primarily to lldb-mi: fixing bugs and filling gaps needed for our use of lldb-mi. How can I start a code review/official submission process?<br>
><br>
><br>
><br>
> Thanks,<br>
><br>
> ChuckR<br>
><br>
> _______________________________________________<br>
> lldb-dev mailing list<br>
> <a href="mailto:lldb-dev@cs.uiuc.edu">lldb-dev@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a><br>
> _______________________________________________<br>
> lldb-dev mailing list<br>
> <a href="mailto:lldb-dev@cs.uiuc.edu">lldb-dev@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a><br>
<br>
<br>
_______________________________________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@cs.uiuc.edu">lldb-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a><br>
</div></div></blockquote></div><br></div>