Skip to content

Conversation

@Pudel-des-Todes
Copy link
Contributor

@Pudel-des-Todes Pudel-des-Todes commented Nov 12, 2025

Proposed fix for Issue#20
I'm not entirely sure what --premium=safety-off does as it is not mentioned in the manual, but unless order of the arguments is important somehow, this should fix open source versions and keep the same arguments in premium.

@danmar
Copy link
Contributor

danmar commented Nov 12, 2025

I'm not entirely sure what --premium=safety-off does

the reason cppcheclipse uses this is that if cppcheck is executed in "safety" mode (default in cppcheck premium) then it will return nonzero exitcode if there are critical errors and then cppcheclipse will fail. with this option cppcheclipse is able to output the error messages..

normally open source does not run in "safety" mode but it could if a cppcheck.cfg with that option is provided.

--no-safety could be an alternative it works both in premium and open source:
https://github.com/danmar/cppcheck/blob/main/cli/cmdlineparser.cpp#L1019
however --no-safety was added last year and will not work on older versions. So well I guess your fix looks reasonable, however I am not an expert on cppcheclipse code.

@danmar danmar changed the title remove premium options from open source version Fix #20 remove premium options from open source version Nov 17, 2025
@danmar danmar merged commit 3505d7e into cppchecksolutions:main Nov 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants