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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits