[PATCH] D20150: clang-rename: fix renaming of field with implicit initializers

Manuel Klimek via cfe-commits cfe-commits at lists.llvm.org
Wed May 11 00:51:26 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:64
@@ +63,3 @@
+      if (Initializer->getSourceOrder() == -1) {
+        // Ignore implicit initializers.
+        continue;
----------------
Add a comment like:
// The source location of implicit initializers is the constructor declaration.

Also: should we add a check that the token of the source location we find actually has the old name?


http://reviews.llvm.org/D20150





More information about the cfe-commits mailing list