[clang-tidy] Disabling checks for multiple lines

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[clang-tidy] Disabling checks for multiple lines

Tom Stellard via cfe-dev
Hi,

currently there is no way to disable clang-tidy checks for a whole
section of code inline in a similar way to // NOLINT for a single line.

I found two references where the topic has already been brought up:

- as a side topic in a review: https://reviews.llvm.org/D54757#1317477

- as a Bugzilla entry: https://bugs.llvm.org/show_bug.cgi?id=32222

Has anyone worked on this so far? I am willing to implement the feature
and bring it upstream. Would such a patch be welcome?

For the syntax, I would use the proposed '// BEGINNOLINT // ENDNOLINT'
with optional list of checks to disable/enable. However, I am open to
other proposals.

Background: I'm currently introducing clang-tidy to a large embedded
code base, where there are quite a few sections for which certain checks
do not apply (platform specific, mixing C and C++, generated code etc.).
It would be very convenient if this feature would be available to us.

Regards,

Tobias

_______________________________________________
cfe-dev mailing list
[hidden email]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev