ilya-biryukov added inline comments.
================ Comment at: clang-tools-extra/clangd/refactor/tweaks/DefineInline.cpp:158 + // i.e: a::Bar<Bar<T>> instead of a::Bar<a::Bar<T>> + printTemplateArgumentList(OS, TSTL.getTypePtr()->template_arguments(), + TD->getASTContext().getPrintingPolicy()); ---------------- Are we trying to replace the whole name? We can avoid this problem by not qualifying template arguments in the first place. I.e. instead of replacing the whole name, including template arguments: ``` [[baz<Bar>]] -> [[::ns::baz<Bar>]] ``` We could simply replace the part before template arguments: ``` [[baz]]<Bar> -> [[::ns::baz]]<Bar> ``` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66647/new/ https://reviews.llvm.org/D66647 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
