[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 Jan 5 02:38:57 PST 2024


================
@@ -71,8 +98,8 @@ llvm::Expected<RenameResult> rename(const RenameInputs &RInputs);
 /// REQUIRED: Occurrences is sorted and doesn't have duplicated ranges.
 llvm::Expected<Edit> buildRenameEdit(llvm::StringRef AbsFilePath,
                                      llvm::StringRef InitialCode,
-                                     std::vector<Range> Occurrences,
-                                     llvm::StringRef NewName);
+                                     std::vector<SymbolRange> Occurrences,
+                                     llvm::SmallVectorImpl<llvm::StringRef> &NewNames);
----------------
kadircet wrote:

why not `llvm::ArrayRef<llvm::StringRef> NewNames` ?

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


More information about the cfe-commits mailing list