The check adds too much nosie on LLVM codebase (especially running clangd
with clang-tidy enabled).
The check seems too aggresive, it will give warnings as long as a struct
has a method, e.g.
struct Foo {
int X;
void f() {};};
Differential D57225
Don't run misc-non-private-member-variables-in-classes check on LLVM. Authored by hokein on Jan 25 2019, 3:27 AM.
Details
The check adds too much nosie on LLVM codebase (especially running clangd The check seems too aggresive, it will give warnings as long as a struct struct Foo { int X;
void f() {};};
Diff Detail
Event TimelineComment Actions it is very noisy (in other projects too) and i would prefer a different warning policy for that check as well (e.g. one warning per class, not for every field it warns against). |