Skip to content

Fix #14471: Improve check: shadowVariable for function argument#8303

Open
ludviggunne wants to merge 8 commits intodanmar:mainfrom
ludviggunne:14471
Open

Fix #14471: Improve check: shadowVariable for function argument#8303
ludviggunne wants to merge 8 commits intodanmar:mainfrom
ludviggunne:14471

Conversation

@ludviggunne
Copy link
Copy Markdown
Collaborator

No description provided.

@firewave
Copy link
Copy Markdown
Collaborator

This should allow us to enable -Wshadow for GCC and -Wshadow-field for Clang.

@ludviggunne ludviggunne force-pushed the 14471 branch 2 times, most recently from 55fc5b2 to a005907 Compare March 30, 2026 08:29
@firewave
Copy link
Copy Markdown
Collaborator

Since this now reports shadowed functions the error ID is misleading. We probably need a new ID.

@ludviggunne
Copy link
Copy Markdown
Collaborator Author

Since this now reports shadowed functions the error ID is misleading. We probably need a new ID.

The error ID does differ actually, e.g. shadowFunction, shadowArgument etc.

@firewave
Copy link
Copy Markdown
Collaborator

The error ID does differ actually, e.g. shadowFunction, shadowArgument etc.

Ah, sorry. I just went by the title of the PR and did not look at the code.

@ludviggunne ludviggunne force-pushed the 14471 branch 3 times, most recently from fc03706 to eeb3eec Compare March 31, 2026 09:17
@sonarqubecloud
Copy link
Copy Markdown

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.

4 participants