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

David Goldman via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 5 09:22:38 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;
----------------
DavidGoldman wrote:

I think it'll be encoded slightly different but VS Code did appear to handle an empty placeholder properly, but I'll have clangd omit encoding it if empty to be safe.

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


More information about the cfe-commits mailing list