Skip to content
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

poor diagnostic for unused function parameter #48021

Closed
rickycodes opened this issue Feb 5, 2018 · 2 comments
Closed

poor diagnostic for unused function parameter #48021

rickycodes opened this issue Feb 5, 2018 · 2 comments
Labels
A-diagnostics Area: Messages for errors, warnings, and lints

Comments

@rickycodes
Copy link

got this warning:

and then following the diagnostic suggestion it broke:

using .. suppresses the warning and works as intended, so perhaps that's what should be suggested?

this was done using notify's raw_watcher

@steveklabnik steveklabnik added the A-diagnostics Area: Messages for errors, warnings, and lints label Feb 5, 2018
@zackmdavis
Copy link
Member

(A duplicate of #47390, this is being fixed in #47922, after which the suggestion would be cookie: _.)

@TimNN
Copy link
Contributor

TimNN commented Feb 6, 2018

Closing as duplicate.

@TimNN TimNN closed this as completed Feb 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-diagnostics Area: Messages for errors, warnings, and lints
Projects
None yet
Development

No branches or pull requests

4 participants