Oliver Stöneberg
Oliver Stöneberg
Should slightly help with the runtime of the selfchecks. We have other checks that make sure we have no completely unused functions so it is safe to enable this.
Still needs some cleanups (I hope I can clean it up - it looks too messy). We now also need a `standards.cpp` so the include does not spill into everything...
To clean up the mess with the inconsistent language I have local changes that prevents the creation of a `TokenList` without a language.
@danmar ``` lib/token.h:1440:10: style: Bit-fields should not be declared [premium-misra-cpp-2023-12.2.1] bool mIsC : 1; ^ ``` This is not really a bitfield but ensuring that these bools do not blow...
Also - is there handling that suppressions for `premium-*` are not being reported as unused with an open-source version (I have not checked the code)?
`callgrind` step in CI: `64,041,451,567` -> `63,581,639,801`
excerpts from selfcheck. before ``` Check time: cli/cmdlineparser.cpp: 1056.33s [...] Check time: lib/checkio.cpp: 221.528s [...] lib/symboldatabase.cpp: 247.606s Check time: lib/tokenize.cpp: 286.758s ``` after ``` Check time: cli/cmdlineparser.cpp: 1027.11s [...] Check...
No feedback in over two weeks - merging.
As suggested by @pfultz2 in https://github.com/danmar/cppcheck/pull/6845#discussion_r1781209227.
@pfultz2 is this still an acceptable change?