[PATCH] D37808: [clang-tidy] Add new hicpp-multiway-paths-covered check for missing branches

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 13 11:23:38 PDT 2017


lebedev.ri added a comment.

In https://reviews.llvm.org/D37808#869879, @JonasToth wrote:

> In https://reviews.llvm.org/D37808#869823, @lebedev.ri wrote:
>
> > What about GNU extension `case 1 ... 3:` ?
>
>
> Strictly speaking, the coding standard (which should be enforced by the patch) requires strict ISO C++11, therefore this extension is not considered directly.




> Does clang support this extension?

It does:

  test.cpp:3:12: warning: use of GNU case range extension [-Wgnu-case-range]
      case 1 ... 3:
             ^



> When this would be a warning, then that case should be added as well.




https://reviews.llvm.org/D37808





More information about the cfe-commits mailing list