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

Fix missing parentheses Fn notation error #72668

Merged
merged 2 commits into from
May 31, 2020
Merged

Fix missing parentheses Fn notation error #72668

merged 2 commits into from
May 31, 2020

Conversation

awoimbee
Copy link
Contributor

Fixes #72611
Well, fixes the error output, I think E0658 is the right error to throw in this case so I didn't change that

@rust-highfive
Copy link
Collaborator

r? @cramertj

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 27, 2020
@estebank
Copy link
Contributor

estebank commented May 29, 2020

r? @estebank

Could you add a test file in src/test/ui/suggestions/?

// run-rustfix
fn apply_to_3<F>(f: F) -> i32 where F: Fn<i32, Output=i32> { f(3) }
//~^ ERROR E0658

Run ./x.py test src/test/ui --stage 1 --keep-stage 1 --bless to create the .stderr and .fixed files.

r=me after that

@rust-highfive rust-highfive assigned estebank and unassigned cramertj May 29, 2020
@awoimbee
Copy link
Contributor Author

awoimbee commented May 30, 2020

r? @estebank

I also improved my code to be more understandable and to get a pretty suggestion in in every case (handling Fn<i32>, Fn<(i32, )>, Fn<(i32, i32, )>, ...) ;)
(not sure what "r=me after that" means, sorry)

@estebank
Copy link
Contributor

@bors r+ rollup

Thanks for this! It looks great.

@bors
Copy link
Contributor

bors commented May 30, 2020

📌 Commit c54d4fe has been approved by estebank

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 30, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request May 30, 2020
Rollup of 13 pull requests

Successful merges:

 - rust-lang#72543 (Account for missing lifetime in opaque and trait object return types)
 - rust-lang#72625 (Improve inline asm error diagnostics)
 - rust-lang#72637 (expand `env!` with def-site context)
 - rust-lang#72650 (Sort sidebar elements)
 - rust-lang#72657 (Allow types (with lifetimes/generics) in impl_lint_pass)
 - rust-lang#72666 (Add -Z profile-emit=<path> for Gcov gcda output.)
 - rust-lang#72668 (Fix missing parentheses Fn notation error)
 - rust-lang#72669 (rustc_session: Cleanup session creation)
 - rust-lang#72728 (Make bootstrap aware of relative libdir in stage0 compiler)
 - rust-lang#72757 (rustc_lexer: Optimize shebang detection slightly)
 - rust-lang#72772 (miri validation: clarify valid values of 'char')
 - rust-lang#72773 (Fix is_char_boundary documentation)
 - rust-lang#72777 (rustdoc: remove calls to `local_def_id_from_node_id`)

Failed merges:

r? @ghost
@bors bors merged commit 65a02f1 into rust-lang:master May 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error[E0658] is broken
5 participants