[PATCH] D112530: [clangd] AddUsing: Fix support for template specializations.

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 26 07:31:01 PDT 2021


kadircet accepted this revision.
kadircet added a comment.
This revision is now accepted and ready to land.

thanks!



================
Comment at: clang-tools-extra/clangd/refactor/tweaks/AddUsing.cpp:284
+        // Remove the template arguments from the name.
+        NameRange = SourceRange(NameRange.getBegin(),
+                                T.getLAngleLoc().getLocWithOffset(-1));
----------------
nit: `NameRange.setEnd(T.getTemplateNameLoc())`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112530/new/

https://reviews.llvm.org/D112530



More information about the cfe-commits mailing list