alexfh added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/readability/FunctionCognitiveComplexityCheck.cpp:230 + explicit FunctionASTVisitor(const bool IgnoreMacros) + : RecursiveASTVisitor<FunctionASTVisitor>(), IgnoreMacros(IgnoreMacros){}; + ---------------- Is a default base class constructor entry necessary in the class initializer list? ================ Comment at: clang-tools-extra/docs/clang-tidy/checks/readability-function-cognitive-complexity.rst:22 + + If set to `true`, the check will flag every basic increment. Default + is `true`. ---------------- What does "flag every basic increment" mean? Can you explain in more detail and maybe add an example? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96281/new/ https://reviews.llvm.org/D96281 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits