Skip to content

donate-cpu-server.py: added "top x" report for unknownMacro - #5466

Merged
danmar merged 1 commit into
cppcheck-opensource:mainfrom
firewave:daca-macro
Sep 20, 2023
Merged

donate-cpu-server.py: added "top x" report for unknownMacro#5466
danmar merged 1 commit into
cppcheck-opensource:mainfrom
firewave:daca-macro

Conversation

@firewave

Copy link
Copy Markdown
Collaborator

No description provided.

@firewave

Copy link
Copy Markdown
Collaborator Author

The most curious one I saw was:

0ad-0.0.26-alpha/libraries/source/cxxtest-4.4/cxxtest/RealDescriptions.cpp:70:101: error: There is an unknown macro here somewhere. Configuration is required. If ; is a macro then please configure it. [unknownMacro]

@danmar

danmar commented Sep 20, 2023

Copy link
Copy Markdown
Contributor

The most curious one I saw was:

That is certainly interesting :-)

@danmar
danmar merged commit 5ae136a into cppcheck-opensource:main Sep 20, 2023
@firewave
firewave deleted the daca-macro branch September 20, 2023 09:25
@firewave

Copy link
Copy Markdown
Collaborator Author

This will possibly contain a lot of "false positives" since some clients are still running a version with the bugged library detection.

Also the messages for ; do not show. Will take a look later.

@firewave

Copy link
Copy Markdown
Collaborator Author

Also the messages for ; do not show. Will take a look later.

Fixed in #5471.

@firewave

Copy link
Copy Markdown
Collaborator Author

There's a long-standing bug in these reports that they also process the messages in the diff: part of the head results leading to wrong counts. I prepared a fix and will test and publish it tomorrow.

@firewave

Copy link
Copy Markdown
Collaborator Author

There's a long-standing bug in these reports that they also process the messages in the diff: part of the head results leading to wrong counts. I prepared a fix and will test and publish it tomorrow.

Fixed in #5474 with some other long-standing issues.

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