lebedev.ri added inline comments.
================
Comment at: clang-tidy/readability/FunctionCognitiveComplexityCheck.cpp:499
+ functionDecl(
+ allOf(isDefinition(), unless(anyOf(isInstantiated(), isImplicit()))))
+ .bind("func"),
----------------
aaron.ballman wrote:
> Since we're restricting this, might as well add deleted and defaulted
> functions to the list of `unless` items (as I think those are still
> definitions).
Good idea!
It seems only the deleted were still matched, not defaulted.
Repository:
rL LLVM
https://reviews.llvm.org/D36836
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
