mitigated -Wunused-function Clang warnings with code added by matchcompiler

[firewave]

/home/user/CLionProjects/cppcheck/cmake-build-relwithdebinfo-clang/lib/build/mc_checkinternal.cpp:7:20: warning: unused function 'match1' [-Wunused-function]
    7 | static inline bool match1(const Token* tok) {
      |                    ^~~~~~

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[firewave]

Using __attribute__((unused)) causes a warning about the functions marked as unused actually being used.

[danmar]

Using attribute((unused)) causes a warning about the functions marked as unused actually being used.

so we are damned if we use it and damned if we don't.. it's not preferable to use it then is it?

[firewave]

so we are damned if we use it and damned if we don't.. it's not preferable to use it then is it?

Not at all since I went for the standards attribute. So if we ever switch to C++17 we can just enable this unconditionally - and use the UNUSED from config.h. Also it is "just" matchcompiled code so nobody ever needs to deal with it except for the compiler.