[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