cppcheck refs #12861 - exit early in `ConditionHandler::traverseCondition()`

Sep 14 '25 01:09 firewave

Analyzing the code from https://trac.cppcheck.net/ticket/12861:

Clang 20 - 11,437,539,974 -> 3,423,215,226 -> 737,254,030

This needs more testing as it might be detrimental in cases which are not affected by excessive astTop() calls.

Sep 14 '25 01:09 firewave

Can we reduce the timeout in the #12847 test with this?

Sep 15 '25 06:09 chrchr-github

Can we reduce the timeout in the #12847 test with this?

Yes. I forgot to add the reference for the code I tested it with.

Sep 15 '25 07:09 firewave

Clang 20 - 11,437,539,974 -> 3,423,215,226 -> 737,254,030

The second drop does not occur. It was a mistake in an intermediate version.

Sep 26 '25 08:09 firewave

Clang 20 - 11,437,539,974 -> 3,423,215,226 -> 737,254,030

The second drop does not occur. It was a mistake in an intermediate version.

There's a way to address it anyways but let's do that in a separate PR since it also might have detrimental effects on other cases.

Sep 26 '25 08:09 firewave

Quality Gate passed

Sep 26 '25 08:09 sonarqubecloud[bot]