KTimeTracker in kdereview
aspotashev at gmail.com
Sun Dec 1 19:08:17 GMT 2019
вс, 1 дек. 2019 г. в 19:43, Albert Astals Cid <aacid at kde.org>:
> El diumenge, 1 de desembre de 2019, a les 13:09:30 CET, Alexander Potashev va escriure:
> > > clang-tidy says that desktops in Task::Task and sessionStartTiMe in Task::init should be const &
> > Fixed these , however I wasn't able to run clang-tidy myself to get
> > the same hints. Tried clang-tidy features in KDevelop and JetBrains'
> > CLion.
> clang-tidy has a million checks and almost none are enabled by default (i think)
> What i do is enable the performance ones manually, i.e. for example this is what we have in the poppler CI
> cmake -DCMAKE_CXX_CLANG_TIDY="clang-tidy;-header-filter=.;-checks=-*,performance-*,bugprone-*,readability-inconsistent-declaration-parameter-name,readability-string-compare,modernize-deprecated-headers,-bugprone-narrowing-conversions,-bugprone-macro-parentheses,-bugprone-suspicious-string-compare,-bugprone-incorrect-roundings,-bugprone-undefined-memory-manipulation;-warnings-as-errors=*" ..
> But this will probably give you lots of errors you don't want, you have to figure out which are th eones you care about.
I managed to get that million of warnings by following this tutorial:
Fixed some more, however some others would be nearly useless nitpicking.
More information about the kde-core-devel