Fix #8089 (feature request: Show date/time of check in HTML report)

[clock999]

This is to fix the bug:
https://trac.cppcheck.net/ticket/8089
Title:
feature request: Show date/time of check in HTML report
Description:
Cppcheck HTML report is a helpful tool to automate checking. It could be extended by the date/time information, when the checking was finished. This date/time information assists developers in fixing issues.

[chrchr-github]

Thanks for your contribution.

A comment from the ticket:

Adding content to XML might break existing solutions. So I suggest to use timestamp of XML file.

Is there a reason you need the timestamp in the XML? In a CI environment, the timestamp will also be available elsewhere. If we implement this at all, it should probably be an option.

[clock999]

Hi CHR,
Thanks for your comments. I think it is ok to use the timestamp of the xml file. With my idea, I just want to only make changes to the script htmlreport/cppcheck-htmlreport. But I don't know how to make the fix to be useful for the CI Env. Could you please give more information about that? Another question is that we can use stdin as the input for htmlreport to get xml, if the input is stdin, I just want to use the current time of the system to be the timestamp of the report. How do you think about that?

[chrchr-github]

Unfortunately I know nothing about the HTMLReport, so maybe @danmar or @firewave would like to chime in.
In any case, there should be a test for this change.

[clock999]

Hi CHR, thanks for your kind reply, do you mean that I need to submit some test code with the commit? As I understand, I need to add some codes under cppcheck/test, right? To merge this fix, what steps or jobs are expected to be done further? Could you please help comment more? Thanks a lot!

[chrchr-github]

Hi CHR, thanks for your kind reply, do you mean that I need to submit some test code with the commit? As I understand, I need to add some codes under cppcheck/test, right? To merge this fix, what steps or jobs are expected to be done further? Could you please help comment more? Thanks a lot!

These tests seem related: https://github.com/danmar/cppcheck/tree/main/test/tools/htmlreport, so you should probably add a test there.

[danmar]

I would prefer this PR title:

Fix #8089 (feature request: Show date/time of check in HTML report)

That will make more sense in the git log later

[danmar]

@clock999 thanks for your contribution. which name would you like that I add in our AUTHORS?
https://github.com/danmar/cppcheck/blob/main/AUTHORS

[clock999]

@clock999 thanks for your contribution. which name would you like that I add in our AUTHORS? https://github.com/danmar/cppcheck/blob/main/AUTHORS

Hi Daniel, please add Wang Yang as my name to the list. Thanks so much!

[clock999]

Hi Daniel, please add Wang Yang as my name to the list. Thanks so much! Best Regards At 2025-03-18 19:58:08, "Daniel Marjamäki" ***@***.***> wrote: @clock999 thanks for your contribution. which name would you like that I add in our AUTHORS? https://github.com/danmar/cppcheck/blob/main/AUTHORS — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: ***@***.***> danmar left a comment (danmar/cppcheck#7305) @clock999 thanks for your contribution. which name would you like that I add in our AUTHORS? https://github.com/danmar/cppcheck/blob/main/AUTHORS — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: ***@***.***>