-
-
Notifications
You must be signed in to change notification settings - Fork 781
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix: warning shadow #1594
Fix: warning shadow #1594
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR's looking good - there are some small changes to be made but then we're happy to merge this.
Good call with the bmp_info
struct and info
-> bmda_probe_info
! We're particularly happy to see the probe type macros sorted out.
Only the BMP is a *BMP* probe, this rename reflects that with a more generic probe type context
This hopefully helps - code readability with a more contextually obvious name - avoiding variable shadowing
b94538d
to
3b16ef0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, merging.
Detailed description
Fix warnings for
-Wshadow=local
See #1590 for context
Your checklist for this pull request
make PROBE_HOST=native
)make PROBE_HOST=hosted
)Closing issues