[clang-tools-extra] [clangd] Let DefineOutline tweak handle member functions (PR #95235)

Julian Schmidt via cfe-commits cfe-commits at lists.llvm.org
Sat Aug 24 08:54:10 PDT 2024


================
@@ -407,10 +431,21 @@ class DefineOutline : public Tweak {
       return !SameFile;
     }
 
-    // Bail out in templated classes, as it is hard to spell the class name,
-    // i.e if the template parameter is unnamed.
-    if (MD->getParent()->isTemplated())
-      return false;
+    for (const CXXRecordDecl *Parent = MD->getParent(); Parent;
+         Parent =
+             llvm::dyn_cast_or_null<const CXXRecordDecl>(Parent->getParent())) {
+      if (auto Params = Parent->getDescribedTemplateParams()) {
----------------
5chmidti wrote:

Please explicitly write the type, it is not clear from the rhs what it is (`const TemplateParameterList *`)

https://github.com/llvm/llvm-project/pull/95235


More information about the cfe-commits mailing list