I've noticed some override statements in the code, so I think
it may be safely assumed that we're at least compiling in C++11
mode.
Fixed override/nullptr/default usages, fixed some spacing
style problems
mart |
I've noticed some override statements in the code, so I think
it may be safely assumed that we're at least compiling in C++11
mode.
Fixed override/nullptr/default usages, fixed some spacing
style problems
No Linters Available |
No Unit Test Coverage |
kirigami-plasmadesktop-integration/plasmadesktoptheme.cpp | ||
---|---|---|
160 ↗ | (On Diff #25123) | if you make it auto the initializer_list will be used not requiring the allocation AFAIU. |
kirigami-plasmadesktop-integration/plasmadesktoptheme.cpp | ||
---|---|---|
160 ↗ | (On Diff #25123) | I didn't quite get it, you mean we can write for (auto state : { QPalette::Active, QPalette::Inactive, QPalette::Disabled }) to get rid of allocation of states array? |
kirigami-plasmadesktop-integration/plasmadesktoptheme.cpp | ||
---|---|---|
160 ↗ | (On Diff #25123) | Yes, or static const auto states = { QPalette::Active, QPalette::Inactive, QPalette::Disabled }; |