Fix #14471: Improve check: shadowVariable for function argument#8303
Fix #14471: Improve check: shadowVariable for function argument#8303ludviggunne wants to merge 8 commits intodanmar:mainfrom
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
ludviggunne
force-pushed
the
14471
branch
3 times, most recently
from
8d1541b to
a5e4899
Compare
|
This should allow us to enable |
ludviggunne
force-pushed
the
14471
branch
2 times, most recently
from
55fc5b2 to
a005907
Compare
|
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. |
Ah, sorry. I just went by the title of the PR and did not look at the code. |
ludviggunne
force-pushed
the
14471
branch
3 times, most recently
from
fc03706 to
eeb3eec
Compare
|
|
Related premium issue: |
| shadowFunction:externals/simplecpp/simplecpp.cpp | ||
| shadowFunction:externals/simplecpp/simplecpp.h |
There was a problem hiding this comment.
Would be great if you could publish an upstream PR which fixes these so we can drop the suppressions on the next bump.
There was a problem hiding this comment.
I checked and we have -Wshadow-field from Clang enabled in simplecpp. And -Wshadow from GCC only reports shadowing in constructors (as expected), so I am curious about these.
5 participants
No description provided.