[clang-tools-extra] Add support for renaming objc methods, even those with multiple selector pieces (PR #76466)

kadir çetinkaya via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 14 23:20:21 PST 2024


================
@@ -696,7 +982,7 @@ renameOutsideFile(const NamedDecl &RenameDecl, llvm::StringRef MainFilePath,
                    FilePath);
     }
     auto RenameEdit =
-        buildRenameEdit(FilePath, AffectedFileCode, *RenameRanges, NewName);
+        buildRenameEdit(FilePath, AffectedFileCode, *RenameRanges, NewNames);
----------------
kadircet wrote:

as discussed yesterday, i was suggesting to move:
```
llvm::SmallVector<llvm::StringRef, 8> NewNames;
NewName.split(NewNames, ":");
```

from lines 928/936 to right before this call to `buildRenameEdit`

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


More information about the cfe-commits mailing list