[PATCH] D54757: [clang-tidy] new check: bugprone-branch-clone

Mon Dec 3 14:35:07 PST 2018

donat.nagy added a comment.

Macro-generated long switches: I cannot access the check results right now, but if I recall correctly, the check assigned these errors to certain lines in the `.inc` files. This means that (1) it is not very easy to to suppress them with `//NOLINT` comments, but (2) it should be relatively easy to filter them by location (ignore all reports coming from `.inc` file).

A quick, tangentially related idea: Would it be useful to implement multiline nolint sections (e.g. `//BEGINNOLINT` – `//ENDNOLINT` or something similar)? This would be helpful for using clang-tidy on projects that contain some automatically generated fragments.

Repository:
  rCTE Clang Tools Extra

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54757/new/

https://reviews.llvm.org/D54757