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



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

Reply via email to