Skip to content

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

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

fixed #11483 (FN unusedFunction for method with inline implementation)#5457
danmar merged 6 commits into
cppcheck-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

firewave commented Sep 18, 2023

Copy link
Copy Markdown
Collaborator Author

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