<div>LGTM, but...</div><div><br></div><div>+      // Warn when a non-static method call, followed by non-static member</div><div>+      // field accesses, followed by a DeclRefExpr.</div><div><br></div><div>Did you accidentally a verb here?</div>
<br><div class="gmail_quote">On Tue, Oct 2, 2012 at 3:36 PM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com" target="_blank">rtrieu@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
More changes to the SelfReferenceChecker with regards to MemberExpr's.  This has better handling of class pointers and not warning when accessing static members.  The basic idea is to treat each group of MemberExpr's together instead of separately visiting each one.  This allows more checking to be done than the previous way of just looking at the MemberExpr above the DeclRefExpr.
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br>