Skip to content

fixed #11483 (FN unusedFunction for method with inline implementation)#5457

Merged
danmar merged 6 commits intocppcheck-opensource:mainfrom
firewave:unused-11483
Sep 20, 2023
Merged

fixed #11483 (FN unusedFunction for method with inline implementation)#5457
danmar merged 6 commits intocppcheck-opensource:mainfrom
firewave:unused-11483

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

No description provided.

@firewave
Copy link
Copy Markdown
Collaborator Author

This restarts #4692.

@firewave
Copy link
Copy Markdown
Collaborator Author

The suppressions work fine in headers. No idea what the initial issue was or maybe it was fixed.

@firewave
Copy link
Copy Markdown
Collaborator Author

firewave commented Sep 18, 2023

I still need to file tickets for the getArrayLength() issues.

https://trac.cppcheck.net/ticket/11999 for the seemingly wrong line for the suppression.
getArrayLength() is actually unused as it is only used in test and conditional code.

@firewave firewave marked this pull request as ready for review September 18, 2023 08:06
@firewave firewave changed the title fixed #11483 fixed #11483 (FN unusedFunction for method with inline implementation) Sep 18, 2023
Comment thread lib/token.h
@firewave
Copy link
Copy Markdown
Collaborator Author

The unreadVariable false negative is already tracked in https://trac.cppcheck.net/ticket/8433.

firewave and others added 6 commits September 20, 2023 13:10
Co-authored-by: chrchr-github <78114321+chrchr-github@users.noreply.github.com>
Co-authored-by: chrchr-github <78114321+chrchr-github@users.noreply.github.com>
Co-authored-by: chrchr-github <78114321+chrchr-github@users.noreply.github.com>
@danmar danmar merged commit f49fedb into cppcheck-opensource:main Sep 20, 2023
@firewave firewave deleted the unused-11483 branch September 20, 2023 13:03
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.

3 participants