Skip to content

donate-cpu-server.py: Fix #9192: Old results are shown in the reports#1941

Merged
versat merged 2 commits into
cppcheck-opensource:masterfrom
versat:donate-cpu-server-ignore-old-results
Jul 1, 2019
Merged

donate-cpu-server.py: Fix #9192: Old results are shown in the reports#1941
versat merged 2 commits into
cppcheck-opensource:masterfrom
versat:donate-cpu-server-ignore-old-results

Conversation

@versat
Copy link
Copy Markdown
Collaborator

@versat versat commented Jul 1, 2019

Trac ticket: https://trac.cppcheck.net/ticket/9192
This commit also fixes that negative values of the elapsed time are
used for calculating total times. These crashes and errors are now
ignored in the time report since there is no useful timing information
in that case.

Tested with a local daca@home server with old and new results.

versat added 2 commits July 1, 2019 10:04
Trac ticket: https://trac.cppcheck.net/ticket/9192
This commit also fixes that negative values of the elapsed time are
used for calculating total times. These crashes and errors are now
ignored in the time report since there is no useful timing information
in that case.

Tested with a local daca@home server with old and new results.
Copy link
Copy Markdown
Collaborator

@danmar danmar left a comment

Choose a reason for hiding this comment

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

looks good to me!

@versat versat merged commit db3284e into cppcheck-opensource:master Jul 1, 2019
@versat versat deleted the donate-cpu-server-ignore-old-results branch July 1, 2019 12:55
@versat
Copy link
Copy Markdown
Collaborator Author

versat commented Jul 1, 2019

@danmar I no longer have write permission for the file. Can you please update the server script.

@danmar
Copy link
Copy Markdown
Collaborator

danmar commented Jul 2, 2019

@versat please try now again..

@versat
Copy link
Copy Markdown
Collaborator Author

versat commented Jul 2, 2019

@versat please try now again..

It works now, thanks.

jubnzv pushed a commit to jubnzv/cppcheck that referenced this pull request Nov 13, 2019
…cppcheck-opensource#1941)

Trac ticket: https://trac.cppcheck.net/ticket/9192
This commit also fixes that negative values of the elapsed time are
used for calculating total times. These crashes and errors are now
ignored in the time report since there is no useful timing information
in that case.

Tested with a local daca@home server with old and new results.
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