[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
Fri Feb 2 05:41:47 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:

can you also move the logic that splits `NewName` into `NewNames` here? with a comment explaining why it's done for objc selectors?

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


More information about the cfe-commits mailing list