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



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

Reply via email to