[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