[clang-tools-extra] [clangd] Let DefineOutline tweak handle member function templates (PR #112345)

[Nathan Ridge via cfe-commits]

================
@@ -443,13 +461,26 @@ class DefineOutline : public Tweak {
         SameFile = true;
 
         // Bail out if the template parameter is unnamed.
+        // FIXME: Is this really needed? It inhibits application on
----------------
HighCommander4 wrote:

Reviewing the discussion 
[here](https://github.com/llvm/llvm-project/pull/95235#discussion_r1730025569), 
the reason is that we have to mention the template parameter as part of the 
out-of-line declaration, e.g.:

Before:
```c++
template <typename T, typename>
struct A {
  void foo() {}
};
```

After:
```c++
template <typename T, typename>
struct A {
  void foo();
};
template <typename T, typename>
void A<T, ???>::foo() {}
```

https://github.com/llvm/llvm-project/pull/112345
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits