ymandel accepted this revision. ymandel added inline comments. This revision is now accepted and ready to land.
================ Comment at: clang/unittests/Tooling/RecursiveASTVisitorTests/Callbacks.cpp:16 +template <typename Derived> +class RecordingVisitorBase : public TestVisitor<Derived> { + bool VisitPostOrder; ---------------- Add class comment? ================ Comment at: clang/unittests/Tooling/RecursiveASTVisitorTests/Callbacks.cpp:17 +class RecordingVisitorBase : public TestVisitor<Derived> { + bool VisitPostOrder; + ---------------- Consider using an enum rather than a bool. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82485/new/ https://reviews.llvm.org/D82485 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits