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

[Roman Lebedev via Phabricator via cfe-commits]

lebedev.ri added a comment.

Ping.

In https://reviews.llvm.org/D36836#877642, @lebedev.ri wrote:

> In https://reviews.llvm.org/D36836#877636, @alexfh wrote:
>
> > > The metric is implemented as per COGNITIVE COMPLEXITY by SonarSource specification version 1.2 (19 April 2017),
> >
> > Err, "I did ask them, and received an answer that it is it can be implemented in clang-tidy" might not be enough.
>
>
> F5372479: original_msg.txt <https://reviews.llvm.org/F5372479>
>
>    Hello Roman,
>   
>    It would not, and I would be happy to be notified once done.
>   
>    Thanks
>   
>    Olivier
>   
>   > On Fri, Jul 28, 2017 at 6:17 PM Roman Lebedev <lebedev.ri at gmail.com> wrote:
>   > Hi.
>   > 
>   > I would like to know, what is the legal status of
>   > https://www.sonarsource.com/docs/CognitiveComplexity.pdf ?
>   > 
>   > If i (or someone else) is to implement the Specification described in
>   > that paper,
>   > which will produce the same results as the Cognitive Complexity in SonarSource
>   > as part of some other static analyzer, for example as a LLVM's clang-tidy check,
>   > would that be illegal thing to do?
>   > 
>   > Roman.
>    -- 
>    Olivier Gaudin | SonarSource
>    CEO & Co-Founder
>    +41 (0)22 510 2424
>    http://sonarsource.com
>   
>
> Might not be enough?


@alexfh please specify, is that enough or not?


Repository:
  rL LLVM

https://reviews.llvm.org/D36836