klimek accepted this revision.
klimek added a comment.
This revision is now accepted and ready to land.

lg


================
Comment at: clang-rename/USRLocFinder.cpp:94
@@ +93,3 @@
+      // that is not to be touched by the rename.
+      
LocationsFound.push_back(DestructorDecl->getLocation().getLocWithOffset(1));
+      if (DestructorDecl->isThisDeclarationADefinition()) {
----------------
I'd add an assert that this actually starts with ~.


http://reviews.llvm.org/D21364



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to