-
Notifications
You must be signed in to change notification settings - Fork 13.8k
Replace iterator structures with impl Trait
.
#48699
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
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
impl Trait
.impl Trait
.
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.
Nice cleanup! Let's go ahead and do the other cases, too!
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 can use impl trait too -- mut f: impl FnMut(NodeIndex, &'a Node<N>) -> bool
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.
same
588f88f
to
d8172b0
Compare
@nikomatsakis comments have been addressed! |
there are travis errors here, I think because we need to remove a now unused type parameter (which was migrated to an |
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.
F is not used
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.
same
d8172b0
to
08a0182
Compare
@bors r+ |
📌 Commit 08a0182 has been approved by |
…omatsakis Replace iterator structures with `impl Trait`. Two commits: * Replace iterator structures with `impl Trait`. * Run rustfmt on `src/librustc_data_structures/graph/mod.rs`.
Two commits:
impl Trait
.src/librustc_data_structures/graph/mod.rs
.