Skip to content

fix #14130: false positive: unusedStructMember with [[maybe_unused]]#7818

Merged
ludviggunne merged 2 commits intodanmar:mainfrom
ludviggunne:14130
Oct 2, 2025
Merged

fix #14130: false positive: unusedStructMember with [[maybe_unused]]#7818
ludviggunne merged 2 commits intodanmar:mainfrom
ludviggunne:14130

Conversation

@ludviggunne
Copy link
Copy Markdown
Collaborator

No description provided.

@ludviggunne ludviggunne force-pushed the 14130 branch 2 times, most recently from bfad711 to 4ad7c65 Compare September 14, 2025 12:23
@firewave
Copy link
Copy Markdown
Collaborator

We should add more tests which cover the added functionality.

Comment thread test/testunusedvar.cpp
@ludviggunne
Copy link
Copy Markdown
Collaborator Author

We should add more tests which cover the added functionality.

Most of the additions came about from failing tests in other components, but I could add some others that are more specific.

@ludviggunne ludviggunne force-pushed the 14130 branch 2 times, most recently from b77f655 to 8f029d3 Compare September 16, 2025 16:17
@firewave
Copy link
Copy Markdown
Collaborator

Most of the additions came about from failing tests in other components, but I could add some others that are more specific.

Good to know. Just looking at the changes this was not evident. So it should be fine then.

@ludviggunne
Copy link
Copy Markdown
Collaborator Author

ludviggunne commented Sep 24, 2025

Oops, I included a commit that shouldn't be here, I'll remove it later tonight.

Edit: Done.

Comment thread lib/tokenize.cpp
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Oct 2, 2025

@ludviggunne ludviggunne merged commit 8b2249c into danmar:main Oct 2, 2025
53 checks passed
@ludviggunne ludviggunne deleted the 14130 branch October 2, 2025 15:20
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