lebedev.ri added a comment.

In https://reviews.llvm.org/D34202#780394, @malcolm.parsons wrote:

> What do you expect for this?
>
>   if (true)
>       if (true)
>           if (true) {
>               int j;
>           }
>


that it is equivalent to

  if (true && true && true) { // 1
    int j;
  }

This was the intent of that option. There is only one compound statement in 
your example. All the docs say that it counts compound statements
https://github.com/llvm-mirror/clang-tools-extra/blob/9fd3636de8d7034ca4640939fefebd9833ef9ea0/docs/clang-tidy/checks/readability-function-size.rst

  .. option:: NestingThreshold
  
      Flag compound statements ...


Repository:
  rL LLVM

https://reviews.llvm.org/D34202



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to