Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | Fix compilation with MSVC | Ivan Donchevskii | 2018-09-20 | 1 | -0/+2 | |
| | | ||||||
* | | Remove more ifdefs for unsupported llvm versions | Sergio Martins | 2018-09-20 | 1 | -33/+5 | |
| | | | | | | | | clazy doesn't build with these versions anymore, so cleanup code | |||||
* | | Remove ifdef that was used to support LLVM <= 3.8 | Sergio Martins | 2018-09-20 | 1 | -7/+0 | |
| | | | | | | | | This is no longer supported | |||||
* | | Fix build with LLVM <= 7 | Sergio Martins | 2018-09-20 | 1 | -1/+1 | |
| | | | | | | | | Caused by the LLVM => 8 fixes | |||||
* | | Move getImmediateExpansionRange to SourceCompatibilityHelpers.h | Sergio Martins | 2018-09-20 | 2 | -10/+10 | |
| | | ||||||
* | | Fix build with LLVM >= 8 | Sergio Martins | 2018-09-20 | 64 | -179/+230 | |
| | | | | | | | | | | | | | | Stmt::getLocStart was renamed to getBeginLoc. Added a level of indirection so we still support LLVM <= 7 BUG: 398851 | |||||
* | | Fix doc path to FHS path usr/share/doc, patch by Scarlett Moore ↵ | Jonathan Riddell | 2018-09-20 | 1 | -1/+1 | |
| | | | | | | | | <sgclark@kde.org> | |||||
* | | Merge branch '1.3' into 1.4 | Sergio Martins | 2018-09-20 | 2 | -15/+16 | |
|\| | ||||||
| * | Require cmake 3.3 as minimum | Sergio Martins | 2018-09-20 | 1 | -1/+1 | |
| | | | | | | | | | | This amends the previous commit. Linking full path needs 3.3 as minimum, probably so CMP0060 is enabled. | |||||
| * | Link against full paths instead of -lfoo | Sergio Martins | 2018-09-20 | 2 | -13/+15 | |
| | | | | | | | | | | Fixes clazy linking against system libraries when it was told to use a LLVM from prefix | |||||
| * | prospective fix for crash with clang-7.0 | Sergio Martins | 2018-09-19 | 2 | -3/+6 | |
| | | | | | | | | | | | | | | | | | | | | This shuffles the code around in hope of avoiding what seems to be a clang bug. The backtrace and valgrind report don't make sense to me, and the crash goes away if building clazy with gcc. CCBUG: 392223 | |||||
| * | Fix running tests with Qt 5.11 | Sergio Martins | 2018-09-19 | 1 | -2/+2 | |
| | | ||||||
| * | Fix doc path to FHS path usr/share/doc, patch by Scarlett Moore ↵ | Jonathan Riddell | 2018-09-19 | 1 | -1/+1 | |
| | | | | | | | | <sgclark@kde.org> | |||||
| * | Fix build with clang 7.0 | Sergio Martins | 2018-09-19 | 1 | -4/+14 | |
| | | | | | | | | (cherry-picked from 06edd15fdda9155b54366d3887a4fbf25cf1d911) | |||||
* | | README: Bump version to 1.4 | Sergio Martins | 2018-09-08 | 1 | -4/+2 | |
| | | ||||||
* | | Add ability to ignore directories via command line | Sergio Martins | 2018-09-08 | 13 | -18/+91 | |
| | | | | | | | | | | | | | | | | | | For ClazyStandalone it's with --ignore-dirs. For clazy it's via the CLAZY_IGNORE_DIRS env variable. Both take a regexp, to make it similar to clang-tidy. BUG: 397885 | |||||
* | | fix test_requested_checks | Eric Lemanisser | 2018-08-30 | 1 | -0/+1 | |
| | | | | | | | | | | | | unneeded-cast is level 3 fixes up 0ba510fa3337007ff6424c97a62ae00eb7ee88ad | |||||
* | | connect-3arg-lambda: Also warn for QMenu::addAction() | Sergio Martins | 2018-08-24 | 5 | -6/+36 | |
| | | ||||||
* | | old-style-connect: Warn for QMessageBox::open(QObject*receiver, char *slot); | Sergio Martins | 2018-08-24 | 5 | -5/+25 | |
| | | ||||||
* | | old-style-connect: Also warn for QMenu::addAction() | Sergio Martins | 2018-08-24 | 4 | -6/+33 | |
| | | ||||||
* | | Properly install man pages into the standard directory | Sergio Martins | 2018-08-21 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | Previously it installs into /usr/man/man1 which is incorrect. It should be /usr/share/man/man1 as specified in FHS[1]. [1]: https://refspecs.linuxfoundation.org/FHS_3.0/fhs/ch04s11.html#usrsharemanManualPages Contributed by: Chocobo1 <Chocobo1@users.noreply.github.com> (KDE's git repo doesn't accept this e-mail) | |||||
* | | unneeded-cast: Move to level 3 | Sergio Martins | 2018-08-18 | 9 | -7/+10 | |
| | | | | | | | | | | There's too many false-positives which are difficult to fix since there are valid uses of "cast to base" and "cast to itself" | |||||
* | | unneeded-cast: Fix detecting casts to itself | Sergio Martins | 2018-08-18 | 2 | -10/+10 | |
| | | ||||||
* | | virtual-signal: Don't warn if signal overrides a non-QObject base | Sergio Martins | 2018-08-18 | 5 | -5/+17 | |
| | | | | | | | | See tests/virtual-signal/non-qobject-base.cpp for an example. | |||||
* | | connect-3arg-lambda: Also warn when using QTimer::singleShot() without context | Sergio Martins | 2018-08-18 | 6 | -3/+54 | |
| | | | | | | | | | | The check name is now inconsistent but would be overkill to create a new check just for QTimer::singleShot(). | |||||
* | | connect-3arg-lambda: Use auto | Sergio Martins | 2018-08-17 | 1 | -2/+1 | |
| | | ||||||
* | | Introduce empty-qstringliteral | Sergio Martins | 2018-08-15 | 13 | -16/+157 | |
| | | | | | | | | | | Suggests to use QString instead of an empty QStringLiteral(). BUG: 393039 | |||||
* | | Introduce raw-environment-function | Sergio Martins | 2018-08-15 | 12 | -0/+136 | |
| | | | | | | | | | | | | | | | | 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 | |||||
* | | old-style-connect: Warn that 1-arg disconnects have to be manually fixed | Sergio Martins | 2018-07-31 | 4 | -7/+31 | |
| | | | | | | | | | | Instead of saying internal error. This probably should be implemented as it's not very hard. | |||||
* | | Remove unneeded debug statement found by Albert | Sergio Martins | 2018-07-31 | 1 | -1/+0 | |
| | | ||||||
* | | Add a logo and an icon | Sergio Martins | 2018-07-28 | 4 | -0/+0 | |
| | | | | | | | | CCBUG: 396904 | |||||
* | | const-signal-or-slot: Improve readme | Sergio Martins | 2018-07-10 | 1 | -1/+6 | |
| | | | | | | | | Clarify what's bad about const signals | |||||
* | | clang-standalone -> clazy-standalone | Albert Astals Cid | 2018-06-06 | 1 | -3/+3 | |
| | | | | | | | | Approved by Sergio on IRC | |||||
* | | update kde tap name | Yurii Kolesnykov | 2018-05-20 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | Reviewers: smartins Reviewed By: smartins Subscribers: smartins Differential Revision: https://phabricator.kde.org/D12364 | |||||
* | | Fix returning-data-from-temporary tests | Sergio Martins | 2018-05-16 | 1 | -1/+2 | |
| | | | | | | | | CCMAIL: Ch.Ehrlicher@gmx.de | |||||
* | | CLazy: fix run_tests.py to work with Qt >= 5.10 | Christian Ehrlicher | 2018-05-16 | 1 | -3/+8 | |
| | | | | | | | | | | | | | | | | | | | | | | | | Summary: CLazy: fix run_tests.py to work with Qt >= 5.10 Test Plan: Try to run with Qt 5.10 Reviewers: smartins Reviewed By: smartins Differential Revision: https://phabricator.kde.org/D12763 | |||||
* | | Make it easier to link to READMEs by removing the level name. | Sergio Martins | 2018-05-15 | 79 | -84/+210 | |
| | | | | | | | | | | | | | | After make install they will still be installed to a level0, 1, 2 sub-folder, as the cmake instructions are generated by generate.py BUG: 394237 | |||||
* | | Actually fix build with clang 7.0 | Sergio Martins | 2018-05-08 | 1 | -1/+1 | |
| | | | | | | | | | | | | Trunk is 7.0, not 8.0 BUG: 393745 | |||||
* | | Fix build with clang 8.0 | Sergio Martins | 2018-05-06 | 1 | -4/+14 | |
| | | | | | | | | BUG: 393745 | |||||
* | | Fix test following type fixing in last commit | Eric Lemanisser | 2018-04-14 | 1 | -1/+1 | |
| | | | | | | | | It fixes up the test following 16af838716a243ed251f30f7f482353d79e3de70 | |||||
* | | Fix typo in warning message: QLatin1String, not QStringLiteral | easyaspi314 (Devin) | 2018-04-10 | 1 | -1/+1 | |
| | | ||||||
* | | Bump minimum clang version to 3.9 | Sergio Martins | 2018-03-31 | 2 | -3/+3 | |
| | | | | | | | | Doesn't compile anymore with 3.8 | |||||
* | | connect-non-signal: Put the QOverload in it's own file | Sergio Martins | 2018-03-28 | 4 | -7/+12 | |
| | | | | | | | | to restrict it to Qt >= 5.7, since it's where QOverload was introduced. | |||||
* | | Bump version, master is 1.4 | Sergio Martins | 2018-03-28 | 1 | -1/+1 | |
| | | ||||||
* | | connect-non-signal: Make it know about QOverload | Sergio Martins | 2018-03-28 | 2 | -2/+12 | |
| | | | | | | | | BUG: 392441 | |||||
* | | Fix detaching methods map: lower/upperBound do have const counter-parts | Sergio Martins | 2018-03-26 | 1 | -3/+1 | |
| | | | | | | | | thanks ericLemanissier | |||||
* | | detaching-member: Dont warn for methods without const counter-parts | Sergio Martins | 2018-03-24 | 6 | -8/+43 | |
| | | | | | | | | Doesn't make much sense to warn for QVector::fill() | |||||
* | | run_tests.py: Also pass the library path when linking | Sergio Martins | 2018-03-23 | 1 | -1/+5 | |
| | | | | | | | | In case you don't have Qt in a system location | |||||
* | | Remove unneeded fixit enums | Sergio Martins | 2018-03-17 | 12 | -86/+45 | |
| | | | | | | | | Most checks with fixits only have 1 fixit, so no need for enum. | |||||
* | | qt-keywords: fix unit-test | Sergio Martins | 2018-03-17 | 1 | -3/+3 | |
| | | | | | | | | order counts |