made `CppCheckExecutor::check_wrapper()` const

[firewave]

[firewave]

We do not detect this ourselves because of the conditional code inside.

I wonder if we could add a flag which does not omit such warnings similar to --emit-duplicates but I guess in this case it is not some global logic you could put a switch on.

[firewave]

We do not detect this ourselves because of the conditional code inside.

This is just an assumption. I do not have actually checked this (yet).

[firewave]

This is just an assumption. I do not have actually checked this (yet).

I confirmed that it is indeed omitted because of the preprocessor checks. But I do not see where the bailout happens in the code (I also wonder if we should write a debug message about the bailout).

[firewave]

But I do not see where the bailout happens in the code

It is the mTokenizer->hasIfdef() call in CheckClass::checkConstFunc().

[firewave]

It is the mTokenizer->hasIfdef() call in CheckClass::checkConstFunc().

I filed some tickets about possible improvements in the handling: https://trac.cppcheck.net/ticket/13740 https://trac.cppcheck.net/ticket/13741

[firewave]

Superseded by #7415.