New lint: unused underscore prefixed argument#17422
New lint: unused underscore prefixed argument#17422clueless-skywatcher wants to merge 4 commits into
unused underscore prefixed argument#17422Conversation
|
Thanks for the pull request, and welcome! You should hear from one of our reviewers after this PR gets at least 2 reviews from the community. Please see the contribution instructions for more information. Namely, in order to ensure the minimum review times lag, PR authors and assigned reviewers should ensure that the review label (
|
This comment has been minimized.
This comment has been minimized.
849d902 to
926c9f2
Compare
|
Lintcheck changes for 926c9f2
This comment will be updated if you push new changes |
There was a problem hiding this comment.
Community review: the second new added report given in lintcheck is for https://docs.rs/backtrace/0.3.73/src/backtrace/symbolize/gimli/libs_dl_iterate_phdr.rs.html#42 - this exactly highlights what I was warned about: the signature of the function comes from being passed as a pointer to https://docs.rs/libc/0.2.186/libc/fn.dl_iterate_phdr.html
a few reports later is https://docs.rs/bytes/1.6.1/src/bytes/bytes.rs.html#1146 - this signature comes from being stored in a function pointer field, https://docs.rs/bytes/1.6.1/src/bytes/bytes.rs.html#118
I didn't check all of the hits, but at least some of these are places where the lint is problematic
Maybe start by excluding anything marked as extern ?
|
☔ The latest upstream changes (possibly #17385) made this pull request unmergeable. Please resolve the merge conflicts. |
.stderrfile)cargo testpasses locallycargo dev update_lintscargo dev fmtchangelog: [
unused_underscore_prefixed_argument]: add lint