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
Copy link
Copy Markdown
Collaborator

danmar commented Sep 20, 2023

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