kadircet added inline comments.
================ Comment at: clang-tools-extra/clangd/ClangdServer.cpp:365 Callback<tooling::Replacements> CB) { - auto Action = [Sel](decltype(CB) CB, std::string File, - std::string TweakID, - Expected<InputsAndAST> InpAST) { + auto Action = [Sel](decltype(CB) CB, std::string File, std::string TweakID, + Expected<InputsAndAST> InpAST) { ---------------- nridge wrote: > kadircet wrote: > > Can you revert this change? > I tried, but clang-format automatically makes the change again. > > Is there some particular clang-format configuration I should apply, so it > produces the same results? I don't currently have any configuration, i.e. I > think it's just reading the .clang-format file in the monorepo root. That's what we use as well(and your formatted version is correct), but unfortunately sometimes people send out changes with wrong formatting and they are hard to notice during review. Generally we try not to touch those lines in irrelevant patches to keep blame clean. If you are running clang-format directly you can instead try clang-format-diff to format only changed lines. But not that crucial. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56370/new/ https://reviews.llvm.org/D56370 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits