================
@@ -205,8 +208,7 @@ void UseUsingCheck::check(const MatchFinder::MatchResult 
&Result) {
   }
 
   if (!ReplaceRange.getEnd().isMacroID()) {
-    const SourceLocation::IntTy Offset =
-        MatchedDecl->getFunctionType() ? 0 : Name.size();
+    const SourceLocation::IntTy Offset = FunctionPointerCase ? 0 : Name.size();
----------------
flovent wrote:

Update: i think that should be done in another PR, because there are also cases 
like:
```
  typedef char x1, (&refArray)();
```
It provides wrong fix-its right now (before PR), although it's not very common 
(multi function pointers seem to me too)

So i will keep the old way here since it will not provide wrong fix-its for 
multi function pointer typedef cases comparing to `getLocForEndOfToken`.

https://github.com/llvm/llvm-project/pull/173751
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to