enabled and fixed performance-enum-size clang-tidy warnings

[firewave]

No description provided.

[firewave]

This totally throws off several premium-only checks...

[firewave]

CC @danmar

[firewave]

Nevermind - it only occurs in the regular version.

[firewave]

I am not able to reproduce these issues locally. I assume it is related to class : std::uint8_t which leads to it no longer being treated as an enum but as a class with inheritance.

[chrchr-github]

Here's a reproducer:

struct S {
    enum class E : std::uint8_t {
        E0
    };
    void f(S::E e) {
        if (e == S::E::E0) {}
    }
    char a[20];
};

[firewave]

Here's a reproducer:

Thanks. That's very much the example I tried. I can reproduce it now on a different system. Maybe there is a Windows/Linux difference. Will check tomorrow.

I filed https://trac.cppcheck.net/ticket/12564 for it.

[firewave]

In the regular selfcheck we only have passedByValue false positives. The additional ones are just with the premium version.

But there is also the following:

cli/processexecutor.cpp:72:1: debug: Scope::checkVariable found variable 'namespace' with varid 0. [varid0]
namespace {
^

lib/mathlib.h:39:5: debug: SymbolDatabase couldn't resolve all user defined types. [debug]
    class value {
    ^

And with the unusedFunction selfcheck:

cli/processexecutor.cpp:77:0: style: The function 'PipeWriter' is never used. [unusedFunction]
        explicit PipeWriter(int pipe) : mWpipe(pipe) {}
^

[firewave]

@danmar After this passes the selfcheck we need an updated premium version which includes all the related fixes so it can pass the CI.

[chrchr-github]

Reduced FP: https://trac.cppcheck.net/ticket/12588

[firewave]

All fixed, thanks. Now we just need an updated premium version.

[firewave]

All fixed, thanks. Now we just need an updated premium version.

The related fixes are:
2b88ca8
3170c17
4503fbf

[firewave]

@danmar We need a 2.14-based premium in the CI for this to land.