<div dir="ltr">On 21 August 2013 08:53, Matt Beaumont-Gay <span dir="ltr"><<a href="mailto:matthewbg@google.com" target="_blank">matthewbg@google.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">


<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div>On Wed, Aug 21, 2013 at 12:05 AM, Nick Lewycky <<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@google.com</a>> wrote:<br>



> The attached patch adjusts the RParenLoc (end loc) used in the<br>
> CXXMemberCallExpr that gets created in the recovery path when we see a<br>
> non-call member access to a member function. (Ie., "foo->method" missing<br>
> parenthesis). This is important to get right, or else the text diagnostic<br>
> printer can assert.<br>
><br>
> The bug is that it places the end sourcelocation at the end of the token<br>
> (reusing the place where it wrote the fixit for inserting the "()") ,<br>
> instead of the beginning of the last token.<br>
<br>
</div></div>Maybe add a comment about why we're not reusing ParenInsertionLoc.<br></blockquote><div><br></div><div>I think that would be excessive. The rule that the end sourcelocation points to the start of the last token is universal across clang. ParenInsertionLoc is the weird one, off pointing at the end of a token, but it's clear why from its excellent name.</div>


<div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
FWIW, I'm not sure I understand the bug from the sentence above. Other<br>
than that, LGTM.<br>
</blockquote></div><br></div><div class="gmail_extra">Okay. Thanks for the review! Committed in r188920.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Nick</div></div>