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

Christian Kandeler via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 26 06:44:17 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()) {
----------------
ckandeler wrote:

Done, but note that auto is not generally used in this project the way you prefer (with the type obvious from the rhs).

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


More information about the cfe-commits mailing list