Summary: | clazy-inefficient-qlist makes no sense anymore with Qt 6 | ||
---|---|---|---|
Product: | [Developer tools] clazy | Reporter: | christian.kandeler |
Component: | general | Assignee: | Sergio Martins <smartins> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | smartins |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/sdk/clazy/commit/d86b9dc6b5ae57e204f6715d2f6113cc196d2d98 | Version Fixed In: | |
Sentry Crash Report: |
Description
christian.kandeler
2021-10-06 14:06:47 UTC
yeah, I'll disable it for Qt6 Git commit d86b9dc6b5ae57e204f6715d2f6113cc196d2d98 by Sergio Martins. Committed on 13/10/2021 at 22:22. Pushed by smartins into branch 'master'. Remove inefficient-qlist check, the issue has been fixed in Qt6 Newer project can use Qt6. Old Qt5 projects shouldn't be doing such invasive changes. If this check was about uncovering bugs instead of reducing performance pessimisation then we could keep it for Qt5 code in production, that's not the case however. M +1 -0 Changelog M +0 -2 CheckSources.cmake M +0 -1 ClazySources.cmake M +0 -2 ClazyTests.cmake M +0 -2 README.md M +0 -12 checks.json D +0 -7 docs/checks/README-inefficient-qlist-soft.md D +0 -5 docs/checks/README-inefficient-qlist.md M +0 -2 readmes.cmake M +0 -4 src/Checks.h D +0 -117 src/checks/inefficientqlistbase.cpp D +0 -63 src/checks/inefficientqlistbase.h D +0 -34 src/checks/level1/inefficient-qlist-soft.cpp D +0 -47 src/checks/level1/inefficient-qlist-soft.h D +0 -35 src/checks/manuallevel/inefficient-qlist.cpp D +0 -46 src/checks/manuallevel/inefficient-qlist.h D +0 -7 tests/inefficient-qlist-soft/config.json D +0 -91 tests/inefficient-qlist-soft/main.cpp D +0 -3 tests/inefficient-qlist-soft/main.cpp.expected D +0 -8 tests/inefficient-qlist/config.json D +0 -55 tests/inefficient-qlist/main.cpp D +0 -4 tests/inefficient-qlist/main.cpp.expected D +0 -4 tests/inefficient-qlist/main.cpp.expected.x86 https://invent.kde.org/sdk/clazy/commit/d86b9dc6b5ae57e204f6715d2f6113cc196d2d98 |