Improved addon execution errorhandling#5451
Improved addon execution errorhandling#5451danmar merged 15 commits intocppcheck-opensource:mainfrom
Conversation
|
Preparations for #4958. |
firewave
force-pushed
the
addon-xx
branch
5 times, most recently
from
d813d87 to
85836bd
Compare
firewave
changed the title
firewave
force-pushed
the
addon-xx
branch
2 times, most recently
from
523efd6 to
036b0e9
Compare
|
|
||
| /** | ||
| * Execute a shell command and read the output from it. Returns true if command terminated successfully. | ||
| * Execute a shell command and read the output from it. Returns exitcode of the command terminated successfully. |
There was a problem hiding this comment.
is this sentence by intention?
Returns exitcode of the command terminated successfully.
Well it needs to be clarified somehow because the exitcode can be nonzero and as I understand it it's always a success exitcode.
| Checks Linux system for [year 2038 problem](https://en.wikipedia.org/wiki/Year_2038_problem) safety. This required [modified environment](https://github.com/3adev/y2038). See complete description [here](https://github.com/danmar/cppcheck/blob/main/addons/doc/y2038.txt). | ||
| + [threadsafety.py](https://github.com/danmar/cppcheck/blob/main/addons/threadsafety.py) | ||
| Analyse Cppcheck dump files to locate threadsafety issues like static local objects used by multiple threads. | ||
| + [naming.py](https://github.com/danmar/cppcheck/blob/main/addons/naming.py) |
There was a problem hiding this comment.
I saw that I forgot to finish the documentation.
|
I updated the documentation a bit but we need to document all addons properly. I added a ticket about that: https://trac.cppcheck.net/ticket/12001. |
…some addon-related tests
…additional message
…nternalError::details`
|
We should clean up the addon folder so it only contains the actual addons.
The tests should be in the main We might also drop some addons which might not provide much value or reat them as examples and thus also moving them out of the main We also need to dogfood this stuff if we ship this as a an official addon (treating it as example might allow us to treat it as "untested and unmaintained" as well). |
danmar
left a comment
danmar
left a comment
There was a problem hiding this comment.
looks good as far as I can tell
2 participants
No description provided.