fixed #14261/#14902 - Regex: added std::regex implementation#7893
fixed #14261/#14902 - Regex: added std::regex implementation#7893firewave wants to merge 1 commit into
std::regex implementation#7893Conversation
|
Might make sense to be able to specify the type via a rule file as well. That would also allow to mix and match the engines depending on required performance/features. Or maybe even only allow it via a file so to not add another CLI option and not having to deal with the left-to-right issues. |
|
This will only add the additional engine. I will also do some performance tests. The follow-up steps are:
|
|
thanks for working on this. 👍 |
@danmar any comment on this? |
|
No comment so I will go with the way to specify the engine via XML only and not providing a CLI option for it. |
d2a9725 to
e3ff0a0
Compare
std::regex implementationstd::regex implementation
I filed some tickets for those: |
|
The documentation on how to use rules is unfortunately some PDFs hosted at Sourceforge so there is nowhere to add the new field. I filed https://trac.cppcheck.net/ticket/14264 about migrating the documentation. |
std::regex implementationstd::regex implementation / added engine to rule XML to specify regex engine
|
Great - Why am I not surprised?! And on macos |
Apparently So that partially derails the plan going forward and makes things more awkward. And I have not even profiled it yet... |
|
|
Coincidentally Visual Studio 2026 was released just a few hours ago and I can confirm that the test passes in it. |
std::regex implementation / added engine to rule XML to specify regex enginestd::regex implementation
|
2add687 to
7276e9a
Compare
47106c2 to
f4da720
Compare
a834862 to
721e9b9
Compare
std::regex implementationstd::regex implementation
9e6eeeb to
4ca01fb
Compare
|
Thanks for working on this. Sorry I didnt reply to your question. An option in the XML sounds fine. My goal is to eventually drop the pcre dependency completely. In gentle manner.. deprecate pcre and tell users to try to migrate to std::regex and say that unless they complain we remove support for pcre in some version 2.x.. |
This is already part of this and the PCRE2 work. |



No description provided.