ccotter marked an inline comment as done.
ccotter added inline comments.
================
Comment at:
clang-tools-extra/clang-tidy/cppcoreguidelines/AvoidCaptureDefaultWhenCapturingThisCheck.cpp:19
+
+namespace clang {
+namespace tidy {
----------------
carlosgalvezp wrote:
> We recently switched to using C++17 nested namespaces for all the clang-tidy
> folder, please update to keep it consistent.
Just curious - I noticed the clang-tidy headers do not use nested namespaces as
part of the recent switch. Are the headers slated to be updated too?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141133/new/
https://reviews.llvm.org/D141133
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
