[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


================
@@ -134,7 +134,7 @@ class ClangdLSPServer : private ClangdServer::Callbacks,
   void onWorkspaceSymbol(const WorkspaceSymbolParams &,
                          Callback<std::vector<SymbolInformation>>);
   void onPrepareRename(const TextDocumentPositionParams &,
-                       Callback<std::optional<Range>>);
+                       Callback<std::optional<PrepareRenameResult>>);
----------------
kadircet wrote:

you can drop the optional now, we either return an Error or PrepareRenameResult

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


More information about the cfe-commits mailing list