aboutsummaryrefslogtreecommitdiffstats
path: root/CheckSources.cmake
Commit message (Collapse)AuthorAgeFilesLines
* qproperty-type-mismatch: Move to manual levelSergio Martins2019-09-031-1/+1
| | | | has many false positives for now
* Introduce qproperty-type-mismatchJean-Michaël Celerier2019-09-031-0/+1
| | | | | | | Warns when any of the functions or variables declared in a Q_PROPERTY have types, arguments or return types differing with the Q_PROPERTY. REVIEW: D22962
* Introduce signal-with-return-valueSérgio Martins2019-06-031-0/+1
| | | | It's a design smell to have signals returning values
* Introduce heap-allocated-small-trivial-typeSergio Martins2019-05-121-0/+1
| | | | | | | Catches heap-allocating small trivially copyable/destructible types. Such as: auto p = new QPoint(); Could contain false-positives, hence going to manual level.
* qvariant-template-instantiation: Move out of level 0, since it has ↵Sergio Martins2019-05-091-1/+1
| | | | | | | | | | | | | | | | false-positives This check has really been noise since its creation. The rate of false-positives don't justify the insignificant compilation performance gains. Moved to manual level now. Currently it has a bug, which doesn't seem possible to solve, as the instantiated template doesn't carry over the information about the qint32 typedef. And the CallExpr doesn't have any template related getters. Also removed this check from the "performance" category in checks.json, since this category is for runtime performance. CCBUG: 407321
* Remove level3 and movel those 5 checks to manual levelSergio Martins2019-03-041-5/+5
| | | | | Doesn't make sense to enable them all at the same time. Each one must be carefully considered, as they are unstable.
* Introduce qrequiredresult-candidatesSergio Martins2018-12-131-0/+1
| | | | | Suggests places where you can use Q_REQUIRED_RESULT. Mostly to be used in your public headers.
* New check: lowercase-qml-type-nameSergio Martins2018-11-181-0/+1
| | | | | | | Warns when QML types registered with qmlRegisterType() or qmlRegisterUncreatableType() don't start with uppercase. It's required by the engine. BUG: 401015
* New check: ifndef-define-typoSergio Martins2018-10-071-0/+1
| | | | | | | | | Tries to find cases where a #define following an #ifndef defines a different but similar name. Possibly indicating a typo. Uses a Levenshtein Distance algorithm to determine the typo likelihood. Disabled by default as it has many false-positives.
* unneeded-cast: Move to level 3Sergio Martins2018-08-181-1/+1
| | | | | There's too many false-positives which are difficult to fix since there are valid uses of "cast to base" and "cast to itself"
* Introduce empty-qstringliteralSergio Martins2018-08-151-0/+1
| | | | | Suggests to use QString instead of an empty QStringLiteral(). BUG: 393039
* Introduce raw-environment-functionSergio Martins2018-08-151-0/+1
| | | | | | | | Suggests qputenv/qgetenv instead of putenv/getenv. Disabled by default. Enable it manually if you need the thread-safety guarantees of the Qt equivalents. BUG: 395303
* Introduce static-pmfSergio Martins2018-03-171-0/+1
| | | | | | | Storing pointer to member functions in static variables is not advisable on MingW. Breaks connect(). CCMAIL: mathias.hasselmann@kdab.com
* Introduce wrong-qevent-castSergio Martins2018-03-111-0/+1
| | | | Warns when QEvents are cast to possibly the wrong class
* Move reserve-candiates to level3Sergio Martins2018-03-041-1/+1
| | | | As it has quite a few false-positives
* Normalize filenames for checksSergio Martins2018-03-041-17/+17
| | | | | Having all of them in the same format makes easier to move checks to another level via script, when needed
* Introducing qhash-with-char-pointer-keySergio Martins2018-03-031-0/+1
| | | | Disabled by default, as there are valid use cases.
* Introduce qt-keywords checkSergio Martins2018-03-031-0/+1
| | | | | | | Although QT_NO_KEYWORDS is useful we can still have a fixit in clazy. Warns for emit, slot, signal and foreach. Disabled by default. FixIt will come next.
* generate.py now also generates CheckSources.cmakeSergio Martins2018-03-021-21/+19
| | | | | This makes it easier to compile only 1 check, useful during development to reduce build times, you just trim down checks.json and generate
* cmake: Move the check filenames into a separate fileSergio Martins2018-03-021-0/+79
To make it easier to build only some checks during development