tschuett added inline comments.
================ Comment at: clang-tools-extra/clangd/refactor/tweaks/AddDoxygenComment.cpp:24 +namespace clang { +namespace clangd { +namespace { ---------------- You can merge this into `namespace clang::clangd`. ================ Comment at: clang-tools-extra/clangd/refactor/tweaks/AddDoxygenComment.cpp:58 +}; + +REGISTER_TWEAK(AddDoxygenComment) ---------------- The LLVM coding style kindly asks you stop the anonymous namespace here. Furthermore, I believe that the `REGISTER_TWEAK` does not work in an anonymous namespace. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140275/new/ https://reviews.llvm.org/D140275 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits