[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:56 PST 2024


================
@@ -1436,6 +1436,14 @@ struct RenameParams {
 };
 bool fromJSON(const llvm::json::Value &, RenameParams &, llvm::json::Path);
 
+struct PrepareRenameResult {
+  /// Range of the string to rename.
+  Range range;
+  /// Placeholder text to use in the editor, if set.
+  std::optional<std::string> placeholder;
----------------
kadircet wrote:

let's drop `optional` to prevent confusion here, there isn't any difference between an empty placeholder and nullopt, right?

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


More information about the cfe-commits mailing list