[PATCH] D21364: clang-rename: implement renaming of classes with a dtor

Manuel Klimek via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 15 01:40:50 PDT 2016


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





More information about the cfe-commits mailing list