Skip to content

refs #4452 - transfer timer results in ProcessExecutor#8521

Merged
firewave merged 1 commit intocppcheck-opensource:mainfrom
firewave:timer-proc
May 5, 2026
Merged

refs #4452 - transfer timer results in ProcessExecutor#8521
firewave merged 1 commit intocppcheck-opensource:mainfrom
firewave:timer-proc

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

@firewave firewave commented May 3, 2026

No description provided.

@firewave firewave changed the title transfer timer results with processes fixed #4452 - transfer timer results with processes May 3, 2026
@firewave firewave changed the title fixed #4452 - transfer timer results with processes refs #4452 - transfer timer results with processes May 3, 2026
@firewave firewave force-pushed the timer-proc branch 2 times, most recently from 15f7364 to 942b513 Compare May 5, 2026 06:49
@firewave firewave changed the title refs #4452 - transfer timer results with processes refs #4452 - transfer timer results in ProcessExecutor May 5, 2026
@firewave firewave marked this pull request as ready for review May 5, 2026 07:13
Comment thread cli/processexecutor.cpp
Comment on lines +376 to +380
// reset so we do not have the data which has already been transferred back
if (mTimerResults)
mTimerResults->reset();
// TODO: how to "reset" mSuppressions?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will be properly addressed in a follow-up.

@firewave firewave merged commit efae8c0 into cppcheck-opensource:main May 5, 2026
79 of 82 checks passed
@firewave firewave deleted the timer-proc branch May 5, 2026 12:08
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.

2 participants