[PATCH] Fixed UseNullptr causes ambiguity

Dmitri Gribenko gribozavr at gmail.com
Fri Mar 8 09:38:58 PST 2013


  Please upload a patch with more context next time, so that phab can display it.


================
Comment at: cpp11-migrate/UseNullptr/NullptrActions.cpp:73-76
@@ -72,4 +72,6 @@
         C->getCastKind() == CK_NullToMemberPointer) {
-      SourceLocation StartLoc = FirstCast->getLocStart();
-      SourceLocation EndLoc = FirstCast->getLocEnd();
+      const Expr *E = FirstCast->getSubExpr();
+
+      SourceLocation StartLoc = E->getLocStart();
+      SourceLocation EndLoc = E->getLocEnd();
 
----------------
Can you simplify the visitor because of this?  Also, the comment on top of CastSequenceVisitor is no longer correct.


http://llvm-reviews.chandlerc.com/D507



More information about the cfe-commits mailing list