[PATCH] D36836: [clang-tidy] Implement readability-function-cognitive-complexity check

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 15 12:21:16 PDT 2017


JonasToth added inline comments.


================
Comment at: clang-tidy/readability/FunctionCognitiveComplexityCheck.cpp:27
+  // Here are all the possible reasons:
+  enum Criteria : unsigned char {
+    None = 0,
----------------
JonasToth wrote:
> i think clarifying which language constructs relative to what criteria would help here.
> 
> with the document next to me here its clear, but i think we shouldn't expect that. a link/refernce to the page would be nice, too.
*relate


Repository:
  rL LLVM

https://reviews.llvm.org/D36836





More information about the cfe-commits mailing list