summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Disable action for non-c++ projectscsa-v3.5.1csa-v3.5.0csa-3.5Nikolai Kosjar2015-08-121-0/+10
| | | | | | | Change-Id: I7ccdd3d364c7761cd1206d526f4e6f7ada133ebf Task-number: QCE-59 Reviewed-by: Andre Poenitz <andre.poenitz@theqtcompany.com> Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* Make license check output more helpful.csa-v3.5.0-rc1Christian Kandeler2015-07-161-1/+5
| | | | | Change-Id: Ia17b5ef5661cd7bd17efbbf0f1985860c220111e Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
* By default, look for the clang executable in Creator's libexec dir.Christian Kandeler2015-07-144-12/+49
| | | | | | | | | It will be part of the installation from 3.5 on. Also look explicitly for "clang-cl" on Windows now, as we do not ship clang.exe. Change-Id: I67e22ed4251791dd59015dd6f9648c7a14d941d7 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
* Handle zero files to processNikolai Kosjar2015-07-092-14/+27
| | | | | | | | The analyzer was stuck in the running mode (busy cursor, progress bar) for zero files. Change-Id: I9e520dc65b6d027b82e6e31043c0401ebb9f3673 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* Fix displaying tab name in the application output panecsa-v3.5.0-beta1Nikolai Kosjar2015-07-071-0/+1
| | | | | | | | | | It was empty due to commit c209eb9fbbbdac177dca0ec5cfb3454fcb8fc262 AnalyerRunControl: Don't store the runconfiguration Change-Id: Ib20ca9ab5b6e4b4fe971e6b9bcba1fba46a3b152 Reviewed-by: Daniel Teske <daniel.teske@theqtcompany.com>
* Factor out license check.Christian Kandeler2015-07-024-20/+51
| | | | | Change-Id: Ifcaab7252239bde1affa8bc55ab38761c0f6e99f Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
* Adapt to API change in main QtC.Christian Kandeler2015-07-011-1/+1
| | | | | Change-Id: I1496b29cc4b3a9802587f67d078b9c3d953f0141 Reviewed-by: Andre Poenitz <andre.poenitz@theqtcompany.com>
* Adapt to changes to run modeEike Ziller2015-06-305-8/+13
| | | | | Change-Id: I9e299d6fa49920ffa98ececb5b96e9512055ba34 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
* Adapt to upstream API change in CompilerOptionsBuilder IIINikolai Kosjar2015-06-291-18/+0
| | | | | | | | excludeDefineLine() can go since the relevant check is now in the base class (again). Change-Id: I572586cc8b52cdd3479b84c677149e3c10a6b804 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* Adapt to upstream API change in CompilerOptionsBuilder IINikolai Kosjar2015-06-292-29/+114
| | | | | Change-Id: I196b69846e22136773cf3cbf463a413d840dc4d2 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* Settings widget: Make sure initial path is also validated.Christian Kandeler2015-06-261-1/+1
| | | | | | | Without this patch, validation kicks in only on user input. Change-Id: I791c45b214e7e171f59c66c7d96f3e85a668427e Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
* qbs build: Soft dependency on LicenseChecker.Christian Kandeler2015-06-261-2/+1
| | | | | Change-Id: Ia74c556155f5c0c0344cabf4ba13b36e31430c68 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Fix license headerNikolai Kosjar2015-06-2636-36/+41
| | | | | Change-Id: I017092613cdec7c00736442c10ca80b3c74f0189 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* Adapt to upstream API change in CompilerOptionsBuilderNikolai Kosjar2015-06-261-22/+10
| | | | | Change-Id: I895e8a32ca8adc1c5f1b4791a3fcc5f18c5078ab Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* Make licensechecker dependency optionalKai Koehne2015-06-253-4/+13
| | | | | Change-Id: Ie867d2da37a507d36caaf16494f996b88ca4e142 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* Adapt to upstream API change in AnalyzerManagerNikolai Kosjar2015-06-231-2/+1
| | | | | Change-Id: Ibc1748db410d99d2b1d58ca3af535c629b0593f0 Reviewed-by: Andre Poenitz <andre.poenitz@theqtcompany.com>
* Adapt to upstream API changeAndré Pönitz2015-06-191-1/+1
| | | | | Change-Id: I33168006b955af224fd914482a042958f6319c4e Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* Adapt to API changes in AnalyzerBase.Christian Kandeler2015-06-182-2/+0
| | | | | Change-Id: Id61d77653c3161f3abc97581f5e9674adf81598b Reviewed-by: Andre Poenitz <andre.poenitz@theqtcompany.com>
* Use native dir separators in user visible pathsAlessandro Portale2015-06-111-4/+5
| | | | | Change-Id: Ifc54be09fae6125a65a016d7030ca1d291e308c2 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* Tests: Minor refactorings and TemporaryCopiedDir checkNikolai Kosjar2015-05-181-4/+5
| | | | | Change-Id: Ia509e2d7229fadca06791f537c462924c32b8add Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* Merge remote-tracking branch 'origin/3.4'Eike Ziller2015-05-075-8/+14
|\ | | | | | | | | | | | | Conflicts: plugins/clangstaticanalyzer/clangstaticanalyzertool.cpp Change-Id: I0362a5bb2a44c2608c2c074df45fffa022446cc4
| * Unit tests: Explicitly check for success from runner.csa-v3.4.2csa-v3.4.1csa-3.4Christian Kandeler2015-05-055-8/+14
| | | | | | | | | | | | | | | | | | | | We will otherwise miss runner failures for any file that does not contribute to the diagnostics count. Note: The tool can also fail during the building stage, but in this case we will get notified by the signal spy timing out. Change-Id: Ia9aa797d658b1752e3da6e08a652ee55868955ba Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
* | Get rid of FancyLineEdit::validateEike Ziller2015-05-061-3/+4
| | | | | | | | | | | | | | Use aggregation instead of inheritance for customization of validation Change-Id: I5a9703bad5bace78e8c3cc1e2c353a734001bd0a Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>
* | Mini refactorings in ClangStaticAnalyzerRunControlNikolai Kosjar2015-05-052-15/+24
| | | | | | | | | | | | | | ...making ClangStaticAnalyzerRunControl::startEngine() a bit shorter. Change-Id: Ie1547d81ba8443d663983bc0c2aa8f342932c338 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* | Run the analyzer within the build environment.Nikolai Kosjar2015-05-055-3/+24
| | | | | | | | | | | | | | | | | | | | This is especially important for clang-cl, which requires the environment set by e.g. vcvars32.bat [1]. [1] http://clang.llvm.org/docs/UsersManual.html#clang-cl Change-Id: If319bb94752bbef9207581c50173dde99af007bc Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* | Extract AnalyzeUnit out of ClangStaticAnalyzerRunControlNikolai Kosjar2015-04-272-23/+20
| | | | | | | | | | | | | | Reduces some noise in the implementation file. Change-Id: I24e81941c1888ba69f6b7f8dcab35956f60ca4e6 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* | Reduce calls to SessionManager:startupProject()Nikolai Kosjar2015-04-272-7/+11
| | | | | | | | | | | | | | Might fix race conditions. Change-Id: I70f7f28c88afe328468332898919b69fc432098b Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* | Merge remote-tracking branch 'origin/3.4'Eike Ziller2015-04-2312-15/+163
|\ \ | |/
| * qbs build: Clear file tags of unit test resources.csa-v3.4.0Christian Kandeler2015-04-161-1/+7
| | | | | | | | | | | | | | Otherwise qbs will try to build the cpp files in there. Change-Id: I8afd776c8e03bd5f2aea5ac254e94168c23290c3 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
| * Add qt-widgets-app unit testNikolai Kosjar2015-04-168-2/+112
| | | | | | | | | | Change-Id: I502075f79d4abf6bd5c0d2080b2f543116499ace Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
| * Fix QTC_CHECK condition in prependWordWidthArgumentIfNotIncluded()Nikolai Kosjar2015-04-161-1/+1
| | | | | | | | | | Change-Id: I8a6a10eab18b8eb0eb1570eb13db91d712fb71ec Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
| * Remove pointless Q_UNUSED()Nikolai Kosjar2015-04-151-2/+0
| | | | | | | | | | Change-Id: I1379864f979dfca1cfd432ec19ad168fbe91ea52 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
| * Set the architecture explicitly ("-m32"/"-m64)Nikolai Kosjar2015-04-152-10/+44
| | | | | | | | | | | | | | | | ...so that clang-cl will predefine size_t to the expected value. Task-number: QCE-52 Change-Id: Icd663c50bae2da8d7bd2d38f19528d57d810c2fa Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* | Merge remote-tracking branch 'origin/3.4'Eike Ziller2015-04-154-6/+11
|\ \ | |/
| * Add missing test dependenciesNikolai Kosjar2015-04-132-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | Adapting to commit 044eeacde5ee1add8b06a8a53b86fd5c6991d488 Load only tested plugins when invoked with -test in the qtcreator repository. Change-Id: If28c6c79fe412e35726567505080e3723092968d Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
| * Fix typo in unit testsNikolai Kosjar2015-04-131-1/+1
| | | | | | | | | | Change-Id: I2e44d2174878211e5dfabc39c80488e5a1848943 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
| * De-QObjectify Settings class.Christian Kandeler2015-04-132-6/+3
| | | | | | | | | | Change-Id: I67d6caff28f1196bec16cfef4dda6354700130f0 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
* | Merge remote-tracking branch 'origin/3.4'Eike Ziller2015-03-112-17/+77
|\ \ | |/
| * Create a dummy run configuration to create our run control from.csa-v3.4.0-rc1Christian Kandeler2015-03-092-1/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Clang Static Analyzer differs from other analyzers in that it does not run a binary produced by the build process, but looks at source files instead. It is therefore completely unrelated to any run configurations that may or may not exist for the project. This has been ignored so far, with these two main consequences: - When running the analyzer, the name of some random run configuration appears in the application output pane, which makes it look to the user as if the corresponding executable has been run, which it has not. - For projects without run configurations (e.g. libraries), analyzing does not work out of the box, which makes no sense conceptually. So we now create our own run special run configuration (not visible in the UI) and run it directly via runRunConfiguration() instead of using the currently active run configuration via runProject(). This fixes both issues listed above. Change-Id: Icc839816f4a1e6f02a0eb2328c536b44f7304807 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
| * Show the status message also while running.Christian Kandeler2015-03-092-16/+38
| | | | | | | | | | | | | | | | So users can see how many issues were found before the analyzer has finished, as well as browse them. Change-Id: I82452441168ecb370e7b2aac137961ebb5b8bfc3 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
* | Merge remote-tracking branch 'origin/3.4'Eike Ziller2015-03-053-16/+14
|\ \ | |/
| * Use new AnalyzerUtils::logToIssuesPane().Christian Kandeler2015-03-051-15/+7
| | | | | | | | | | Change-Id: I9820a81f67c06b7bb54720772467263bfa9fc8f3 Reviewed-by: Andre Poenitz <andre.poenitz@theqtcompany.com>
| * Add changes-3.4.0csa-v3.4.0-beta1Eike Ziller2015-03-021-0/+7
| | | | | | | | | | | | Change-Id: Idfd70d71a3290f4c3e9968f2882e7a671f8995d8 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com> Reviewed-by: Riitta-Leena Miettinen <riitta-leena.miettinen@theqtcompany.com>
| * Remove experimental stateEike Ziller2015-03-021-1/+0
| | | | | | | | | | Change-Id: I8b66e228dff39db37b66270fec39ca526b6373a0 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* | Remove now-unneeded PathChooser subclass.Christian Kandeler2015-03-036-88/+14
| | | | | | | | | | Change-Id: Ic7cbe4566159675ae9e776d723ed213c1e7c71ec Reviewed-by: Andre Poenitz <andre.poenitz@theqtcompany.com>
* | Merge remote-tracking branch 'origin/3.4'Eike Ziller2015-03-0224-45/+967
|\ \ | |/
| * Make use of the "issues" pane when errors occur during analyzing.Christian Kandeler2015-02-271-6/+24
| | | | | | | | | | | | | | | | | | | | | | | | They can otherwise easily get lost in the Application output pane. Policy is as follows: - Failure to analyze a specific file is considered a warning. - If no file could be successfully analyzed, we add an error and pop up the issues pane. This approach is neither too noisy nor too quiet. Change-Id: Ifc577a215006a6a565eee7de5099bd690427f7de Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
| * Allow users to suppress diagnostics.Christian Kandeler2015-02-2716-10/+808
| | | | | | | | | | | | | | | | | | | | | | | | This patch deals with what is likely the most common use case: Filtering specific messages at a particular location. The current granularity is essentially per-file (and per-function, where possible), which seems more useful than taking line numbers into account, as that would not be robust with regards to code changes elsewhere in the file. We can fine-tune this if the need arises. Change-Id: I4e9b2671fa199339cc3b995953d072b840cd3205 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
| * Clear status message on startNikolai Kosjar2015-02-271-0/+1
| | | | | | | | | | Change-Id: I82e94f0426796473b35dd2a534a4fd7c5aab2667 Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
| * Don't attempt to analyze using icecc masquerading as clang.Christian Kandeler2015-02-268-10/+106
| | | | | | | | | | | | | | Also warn the user in the settings page against this. Change-Id: I4dbae953aa85f8dbdc9baa8dd0fda8ff0da45b76 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
View Lorry Controller Status