fixed #14000 - deprecated the usage of rules

[firewave]

No description provided.

[firewave]

This is for the case when you compile it through the provided Visual Studio project which utilized neither CMake nor make.

[danmar]

hmm.. personally I really like to remove the rules feature. However I've wanted to get rid of rules for many years but when I have asked around there was always some users that strongly wanted to keep them..

It seems people don't want to touch any python code.

In theory we could add an addon that more or less simulates the cppcheck rules functionality. It can load the rules files and run the provided regular expressions.. I don't envision that it can be 100% compliant but it can provide similar functionality.

[danmar]

@pfultz2 @orbitcowboy as far as I remember you have used rules..

[pfultz2]

We do use some rules here: https://github.com/pfultz2/cppcheck-rules/blob/master/rules/UseStlAlgorithm/rule.xml

Its not really straightforward to translate those to addons.

[pfultz2]

In theory we could add an addon that more or less simulates the cppcheck rules functionality. It can load the rules files and run the provided regular expressions.. I don't envision that it can be 100% compliant but it can provide similar functionality.

This would be slower, but not just because of the regex in python. The slowest part of the addon is loading the xml files, unfortunately.

It would be nice if we saved to protobuf(or something similiar) as it can build bindings in many different languages, and the loading would be much quicker.

[danmar]

It would be nice if we saved to protobuf(or something similiar) as it can build bindings in many different languages, and the loading would be much quicker.

I agree. It is possible that Cppcheck Solutions can look into this someday but it is not very high on the prio list right now unfortunately.

[sonarqubecloud]

Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[firewave]

So somebody please file tickets for potential future work items and I will close the PR then.

[danmar]

I created this ticket to drop the PCRE dependency:
https://trac.cppcheck.net/ticket/14087

[firewave]

I created this ticket to drop the PCRE dependency: https://trac.cppcheck.net/ticket/14087

There's also the points about protobuf/slow XML loading.

FYI I have a follow-up to the refactoring which adds std::regex support so we can disable PCRE out of.

[danmar]

There's also the points about protobuf/slow XML loading.

I have created this ticket about changing the format of the dumpfiles:
https://trac.cppcheck.net/ticket/12831

[danmar]

FYI I have a follow-up to the refactoring which adds std::regex support so we can disable PCRE out of.

👍