clang-tidy-review icon indicating copy to clipboard operation
clang-tidy-review copied to clipboard

Published 20 hours ago • verified publisher icon ZedThree

Option to set `--extra-arg`

Open bwrsandman opened this issue 2 years ago • 2 comments

I'm running into an issue where I'm getting an irrelevant warning on my source code that happens when a define is not set. This is a check to prevent inclusion of certain headers without the define.

When compiling, the correct cpp files have the define but clang-tidy will not have them when analyzing the header itself.

The solution to this is adding --extra-arg=-DMY_DEFINE to the command line. In clang-tidy 17, you can set this in the config with ExtraArgs but I'm currently on 15.

Either way, I don't see such an option in clang-tidy-review

Note that multiple --extra-arg can be set one after the other.

bwrsandman avatar Jul 07 '23 12:07 bwrsandman

This option would be very useful!

BotellaA avatar Jul 26 '23 15:07 BotellaA

Sounds good. Not sure I'll have much time to implement this myself for awhile, but PRs always welcome!

ZedThree avatar Jul 28 '23 07:07 ZedThree