aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Also include manuallevel checks in clazy --explain <check>1.4-basedSergio Martins2018-10-141-1/+1
| | | | | It wasn't finding them before Patch by Allen
* empty-qstringliteral: Improve readmeSergio Martins2018-10-141-1/+4
| | | | QStringLiteral().isNull() is false
* Fix unintended octal valueSergio Martins2018-10-091-1/+1
| | | | CCBUG: 399544
* empty-qstringliteral: Don't warn for uic generated filesSergio Martins2018-10-094-1/+69
| | | | | | | Since 5.12 this is no longer an issue, since QStringLiteral was abolished from ui_*.h files. BUG: 399544
* Fix Utils::filenameForLoc(loc) for when loc is a macroIdSergio Martins2018-10-091-1/+4
|
* Add QtUtils::isUIFile(loc)Sergio Martins2018-10-091-0/+6
| | | | Returns true for ui_*.h files
* Improve readmes for the new 1.4 checksSergio Martins2018-10-079-38/+30
|
* skipped-base-method: Improve READMESergio Martins2018-10-071-2/+5
|
* skipped-base-method: Fix typo in readmeSergio Martins2018-10-041-1/+1
|
* skipped-base-method: Fix typo in readmeSergio Martins2018-10-041-1/+1
|
* Fix checkname in changelogSergio Martins2018-10-031-1/+1
| | | | It's skipped-based-method, not skipped-base-class
* docs: add empty line between =encoding and =head1 NAMEPino Toscano2018-09-301-0/+1
| | | | This seems to make the resulting man page more valid.
* Fix clazy --list not listing the correct doc directorySergio Martins2018-09-301-3/+3
| | | | | The doc directory changed a few weeks ago but the clazy script wasn't adjusted
* Don't run tests for checks that clazy wasn't compiled withSergio Martins2018-09-261-0/+8
| | | | BUG: 399090
* Don't register qstring-from-literal if ast matchers are disabledSergio Martins2018-09-262-1/+4
| | | | | | | Next we need a way for the unit-tests to know that this check isn't available CCBUG: 399090
* Changelog: Add date of 1.4 releasev1.4Sergio Martins2018-09-231-1/+1
|
* README: Minimum LLVM for clazy 1.4 is now 4.0Sergio Martins2018-09-231-1/+1
|
* tests: Fixes usages of minimum_qt_versionSergio Martins2018-09-239-10/+10
| | | | 5.5 is 50500, not 550
* docker: Add ubuntu-18.04Sergio Martins2018-09-231-0/+6
|
* README: Remove reference to ubuntu's versionSergio Martins2018-09-231-1/+1
| | | | The instructions also apply to later versions, so only say "ubuntu"
* tests: connect-3arg-lambda test will only run on clang >= 5.0 nowSergio Martins2018-09-231-0/+1
| | | | Now worth spending time fixing it for old clang 4.0
* tests: Require Qt >= 5.8 for fully-qualified-moc-typesSergio Martins2018-09-231-0/+1
| | | | | The bundled .moc file has macros that were only introduced in 5.8 (QT_WARNING_DISABLE_DEPRECATED)
* tests: rule-of-three requires clang > 4.0Sergio Martins2018-09-231-0/+1
| | | | | It's failing with 4.0. Will bump master to require 5.0 for all checks.
* docker: Fix llvm's 6 pathSergio Martins2018-09-231-1/+3
|
* docker: Also test llvm 5.0 and llvm 4.0Sergio Martins2018-09-231-0/+16
|
* Fix crash in old-style-connect fixitsSergio Martins2018-09-231-1/+22
| | | | BUG: 388925
* build-clazy.sh: Remove unneeded checksSergio Martins2018-09-231-15/+0
|
* run_tests.py: Silence warning about missing config.jsonSergio Martins2018-09-231-1/+1
| | | | Not all folders have unit-tests. For example docker/
* Cleanup the list of docker containersSergio Martins2018-09-221-14/+8
| | | | | | | | | Removed ubuntu 16.04 since llvm 3.8 isn't spported anymore. Removed archlinux, since I already test it on it as my native env, will add some other distro in its place. Added a comment property just to list the llvm and Qt version, just for info
* Fix typo in bash scriptSergio Martins2018-09-221-3/+3
|
* tests: Don't run qcolor-from-literal when using the crash workaroundSergio Martins2018-09-223-5/+22
| | | | This check requires AST matches, which are disabled on debian+llvm7
* run_tests.py: Add --exclude argumentSergio Martins2018-09-221-1/+4
| | | | Receives a comma separated list of checks to not run
* Merge branch '1.3' into 1.4Sergio Martins2018-09-223-3/+22
|\
| * Workaround crash with LLVM 7upstream/1.3Sergio Martins2018-09-222-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If clazy is compiled with clang instead of gcc it might crash with: ==10637== Process terminating with default action of signal 11 (SIGSEGV): dumping core ==10637== Access not within mapped region at address 0x8 ==10637== at 0x19CDD8C: clang::ast_matchers::MatchFinder::MatchFinder(clang::ast_matchers::MatchFinder::MatchFinderOptions) (in /usr/lib/llvm-7/bin/clang) ==10637== by 0x9D75670: ClazyASTConsumer (Clazy.cpp:62) ==10637== by 0x9D75670: ClazyASTAction::CreateASTConsumer(clang::CompilerInstance&, llvm::StringRef) (Clazy.cpp:183) ==10637== by 0x9E29ED: clang::FrontendAction::CreateWrappedASTConsumer(clang::CompilerInstance&, llvm::StringRef) (in /usr/lib/llvm-7/bin/clang) ==10637== by 0x9E8FCA: clang::FrontendAction::BeginSourceFile(clang::CompilerInstance&, clang::FrontendInputFile const&) (in /usr/lib/llvm-7/bin/clang) ==10637== by 0x9AE3D5: clang::CompilerInstance::ExecuteAction(clang::FrontendAction&) (in /usr/lib/llvm-7/bin/clang) ==10637== by 0xA8C9FA: clang::ExecuteCompilerInvocation(clang::CompilerInstance*) (in /usr/lib/llvm-7/bin/clang) ==10637== by 0x5822C7: cc1_main(llvm::ArrayRef<char const*>, char const*, void*) (in /usr/lib/llvm-7/bin/clang) ==10637== by 0x571ACC: main (in /usr/lib/llvm-7/bin/clang) After debugging clazy and clang's code I couldn't find anything wrong with it. Valgrind's output doesn't make much sense, and simply compiling the Clazy.cpp translation unit with gcc instead of clang makes the crash go away and valgrind's output is clean. I'm assuming debian's LLVM was built with gcc and building clazy with clang will have some sort of incompatibility, or maybe it's simply a clang bug. The downside of this workaround is that qcolor-literal check will be disabled. Next step will be producing a minimal test case and reporting to LLVM. BUG: 392223 CCMAIL: Woebbeking@kde.org
* | build-clazy.sh: Don't checkout an old branchSergio Martins2018-09-221-1/+1
| | | | | | | | Now we checkout what we fetched
* | test_docker.py: Fix passing --branch argumentSergio Martins2018-09-221-0/+2
| |
* | run_tests.py: Print compiler output when test fails to compileSergio Martins2018-09-221-0/+10
| |
* | docker: Fix llvm paths on debianSergio Martins2018-09-221-3/+3
| | | | | | | | It's llvm-7/ not llvm-7.0/
* | test_docker.py: Fix typoSergio Martins2018-09-221-1/+1
| |
* | docker: Test on debian with clang 7.0 tooSergio Martins2018-09-221-1/+7
| |
* | test_docker.py: Add llvm_root property to jsonSergio Martins2018-09-222-1/+6
| | | | | | | | | | | | On debian llvm-config is called llvm-config-X and there's symlinks pointing to the correct X. By setting LLVM_ROOT env variable FindLLVM.cmake will find the correct one
* | test_docker.py: Allow to test two clang version on the same containerSergio Martins2018-09-222-8/+18
| | | | | | | | | | | | debian container for example can have many clang versions, so in our conf.json "name" is now the ID. Different names can use the same docker url now.
* | test_docker.py: Default prefix to /usrSergio Martins2018-09-222-4/+10
| | | | | | | | That's where clang is in most distros.
* | test_docker.py: Execute the build script from our repoSergio Martins2018-09-221-1/+1
| | | | | | | | The one in /usr/bin will be deleted from the containers
* | test_docker.py: Allow to specify a single docker container to runSergio Martins2018-09-221-3/+12
| |
* | docker: pass the prefix to the build scriptSergio Martins2018-09-222-2/+5
| | | | | | | | | | Some distros support multiple versions of clang, each in their own prefix.
* | Add the script thats called by the docker container to build clazySergio Martins2018-09-221-0/+22
| | | | | | | | | | Previsouly it was in the container's /usr/bin/, so got out of sync with other containers when edited.
* | Make test_build_on_distros.sh configurable via a json fileSergio Martins2018-09-223-21/+77
| | | | | | | | | | | | | | By rewriting it in python. "iamsergio/clazy-" is not hardcoded in the script anymore. It can be replaced in the json file.
* | Fix linking when using MSVCSergio Martins2018-09-211-11/+1
| | | | | | | | | | the string replace was transforming the list back into a string fix is simply removing that code as it builds fine without it
* | Simplify cmake macroSergio Martins2018-09-211-7/+5
| |
generated by cgit v1.2.3 (git 2.25.1) at 2024-05-13 02:56:19 +0000