[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