[PATCH] D38882: [clang-rename] Add function unit tests.

[Haojian Wu via Phabricator via cfe-commits]

hokein added inline comments.


================
Comment at: lib/Tooling/Refactoring/Rename/USRLocFinder.cpp:469
+          // name.
+          // FIXME: Consider using using-decls to shorten the namespace
+          // qualifers.
----------------
ioeric wrote:
> The current behavior looks fine to me, but I'm not sure if this is a right `FIXME`? Basically, I think we want to be able to get the real `DeclContext` for the symbol reference here right?
I'm fine on removing it, as this is somewhat mentioned above. Done.


https://reviews.llvm.org/D38882