| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
It was empty due to
commit c209eb9fbbbdac177dca0ec5cfb3454fcb8fc262
AnalyerRunControl: Don't store the runconfiguration
Change-Id: Ib20ca9ab5b6e4b4fe971e6b9bcba1fba46a3b152
Reviewed-by: Daniel Teske <daniel.teske@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: Ifcaab7252239bde1affa8bc55ab38761c0f6e99f
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: I1496b29cc4b3a9802587f67d078b9c3d953f0141
Reviewed-by: Andre Poenitz <andre.poenitz@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: I9e299d6fa49920ffa98ececb5b96e9512055ba34
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
| |
Change-Id: I196b69846e22136773cf3cbf463a413d840dc4d2
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
|
|
|
|
|
|
| |
Without this patch, validation kicks in only on user input.
Change-Id: I791c45b214e7e171f59c66c7d96f3e85a668427e
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: Ia74c556155f5c0c0344cabf4ba13b36e31430c68
Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: I017092613cdec7c00736442c10ca80b3c74f0189
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: I895e8a32ca8adc1c5f1b4791a3fcc5f18c5078ab
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: Ie867d2da37a507d36caaf16494f996b88ca4e142
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: Ibc1748db410d99d2b1d58ca3af535c629b0593f0
Reviewed-by: Andre Poenitz <andre.poenitz@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: I33168006b955af224fd914482a042958f6319c4e
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: Id61d77653c3161f3abc97581f5e9674adf81598b
Reviewed-by: Andre Poenitz <andre.poenitz@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: Ifc54be09fae6125a65a016d7030ca1d291e308c2
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: Ia509e2d7229fadca06791f537c462924c32b8add
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
|\
| |
| |
| |
| |
| |
| | |
Conflicts:
plugins/clangstaticanalyzer/clangstaticanalyzertool.cpp
Change-Id: I0362a5bb2a44c2608c2c074df45fffa022446cc4
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| | |
Use aggregation instead of inheritance for customization of validation
Change-Id: I5a9703bad5bace78e8c3cc1e2c353a734001bd0a
Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| | |
...making ClangStaticAnalyzerRunControl::startEngine() a bit shorter.
Change-Id: Ie1547d81ba8443d663983bc0c2aa8f342932c338
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| | |
Reduces some noise in the implementation file.
Change-Id: I24e81941c1888ba69f6b7f8dcab35956f60ca4e6
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| | |
Might fix race conditions.
Change-Id: I70f7f28c88afe328468332898919b69fc432098b
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
|\ \
| |/ |
|
| |
| |
| |
| |
| |
| |
| | |
Otherwise qbs will try to build the cpp files in there.
Change-Id: I8afd776c8e03bd5f2aea5ac254e94168c23290c3
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
|
| |
| |
| |
| |
| | |
Change-Id: I502075f79d4abf6bd5c0d2080b2f543116499ace
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
| |
| |
| |
| |
| | |
Change-Id: I8a6a10eab18b8eb0eb1570eb13db91d712fb71ec
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
| |
| |
| |
| |
| | |
Change-Id: I1379864f979dfca1cfd432ec19ad168fbe91ea52
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
...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>
|
|\ \
| |/ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| | |
Change-Id: I2e44d2174878211e5dfabc39c80488e5a1848943
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
| |
| |
| |
| |
| | |
Change-Id: I67d6caff28f1196bec16cfef4dda6354700130f0
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
|
|\ \
| |/ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
|\ \
| |/ |
|
| |
| |
| |
| |
| | |
Change-Id: I9820a81f67c06b7bb54720772467263bfa9fc8f3
Reviewed-by: Andre Poenitz <andre.poenitz@theqtcompany.com>
|
| |
| |
| |
| |
| |
| | |
Change-Id: Idfd70d71a3290f4c3e9968f2882e7a671f8995d8
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
Reviewed-by: Riitta-Leena Miettinen <riitta-leena.miettinen@theqtcompany.com>
|
| |
| |
| |
| |
| | |
Change-Id: I8b66e228dff39db37b66270fec39ca526b6373a0
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
| |
| |
| |
| |
| | |
Change-Id: Ic7cbe4566159675ae9e776d723ed213c1e7c71ec
Reviewed-by: Andre Poenitz <andre.poenitz@theqtcompany.com>
|
|\ \
| |/ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| | |
Change-Id: I82e94f0426796473b35dd2a534a4fd7c5aab2667
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| | |
Also warn the user in the settings page against this.
Change-Id: I4dbae953aa85f8dbdc9baa8dd0fda8ff0da45b76
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
There is no reason why the status of the analyzer run should disappear
after a few seconds.
Change-Id: I16c21f05a03ac31f489312fbe56373a531ecc5d5
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
|
| |
| |
| |
| |
| | |
Change-Id: I2985b1be505f9aa43f2a8615a8dfcafaeb991e21
Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
|
| |
| |
| |
| |
| | |
Change-Id: I3870488ff2052465508c23d429dcb2d25a249857
Reviewed-by: Andre Poenitz <andre.poenitz@theqtcompany.com>
|
| |
| |
| |
| |
| |
| | |
Change-Id: I100e75c526ff8609849526c3317a6d0a1658176b
Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>
|