<div dir="ltr">One patch per bullet point in the original patch's commit message seems to be the most logical thing to me.<div><br><div>-- Sean Silva</div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Fri, Sep 27, 2013 at 12:24 AM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">Sean Silva <<a href="mailto:silvas@purdue.edu">silvas@purdue.edu</a>> writes:<br>
> I replied to your first ping on Jul 22; this patch doesn't seem to address my<br>
> comments ("Please split this into separate commits for each logical change")<br>
<br>
</div>I missed that message somehow. Sorry about that. How's two commits<br>
sound, ie:<br>
<br>
- Remove the note about Identifiers used for C++ operators<br>
- Document CXXLiteralOperatorName and CXXUsingDirective<br>
<br>
or would you consider the two new DeclarationNames to be different<br>
logical changes as well?<br>
</blockquote></div><br></div>