[clang-tools-extra] [clangd] forward clang-tidy's readability-identifier-naming fix to textDocument/rename (PR #78454)

Tom Praschan via cfe-commits cfe-commits at lists.llvm.org
Sun Feb 18 03:13:44 PST 2024


================
@@ -648,6 +649,27 @@ tweakSelection(const Range &Sel, const InputsAndAST &AST,
   return std::move(Result);
 }
 
+// Some fixes may perform local renaming, we want to convert those to clangd
+// rename commands, such that we can leverage the index for more accurate
+// results.
+std::optional<ClangdServer::CodeActionResult::Rename>
+TryConvertToRename(const Diag *Diag, const ClangdServer::DiagRef &DR,
----------------
tom-anders wrote:

hah :D 

At work we're using Google style, so functions are CamelCase and variables snake_case. Switching back and forth between Google and LLVM style is so confusing

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


More information about the cfe-commits mailing list