vmiklos added inline comments.
================
Comment at: clang-rename/USRLocFinder.cpp:64
@@ +63,3 @@
+ if (Initializer->getSourceOrder() == -1) {
+ // Ignore implicit initializers.
+ continue;
----------------
klimek wrote:
> 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?
I don't know off the top of my head a situation where this is needed, but sure,
sounds like a useful safety check. I'll have a look.
Repository:
rL LLVM
http://reviews.llvm.org/D20150
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
