update array missing IntoIterator msg#82626
Conversation
rust-highfive
added
the
S-waiting-on-review
|
I believe https://doc.bccnsoft.com/docs/rust-1.36.0-docs-html/unstable-book/language-features/on-unimplemented.html is still up-to-date. I'm not sure I follow what you are asking for, but it's likely not available (unless you modify |
There was a problem hiding this comment.
Any reason not to suggest the &?
There was a problem hiding this comment.
hmm, i wasn't sure how to explain that in a concise way, as adding & changes the element type.
Do you have an idea for a second label here?
There was a problem hiding this comment.
Looking at the array of ranges cases, it might make sense for us to suggest that maybe they didn't want the outer [], just the range. That seems to be the most likely intent behind the code as written.
There was a problem hiding this comment.
It seems like we already had a check for those cases https://github.com/rust-lang/rust/pull/82626/files#diff-c552b2bf18fb3212fe93dd601ce487badf5e50faf9aeed4db7b0f6c22571ef01L28-L33
Maybe at some point inference broke and instead of Idx we got {integer}?
There was a problem hiding this comment.
AH! It broke not because of that, but because of 07e7823 and there being a single RangeInclusive but multiple Range structs in the std. rustc_on_unimplemented has a bug, it should always be using the full path for its filtering.
lcnr
force-pushed
the
encode_with_shorthandb
branch
from
1156478 to
77bfa69
Compare
|
☔ The latest upstream changes (presumably #75384) made this pull request unmergeable. Please resolve the merge conflicts. |
|
r=me after fixing the merge conflict (sorry for not approving earlier) |
lcnr
force-pushed
the
encode_with_shorthandb
branch
from
77bfa69 to
5ac917d
Compare
|
@bors r=estebank |
|
📌 Commit 5ac917d has been approved by |
bors
added
S-waiting-on-bors
…bank update array missing `IntoIterator` msg fixes rust-lang#82602 r? `@estebank` do you know whether we can use the expr span in `rustc_on_unimplemented`? The label isn't too great rn
…bank update array missing `IntoIterator` msg fixes rust-lang#82602 r? ``@estebank`` do you know whether we can use the expr span in `rustc_on_unimplemented`? The label isn't too great rn
Rollup of 8 pull requests Successful merges: - rust-lang#81351 (combine: stop eagerly evaluating consts) - rust-lang#82525 (make unaligned_references future-incompat lint warn-by-default) - rust-lang#82626 (update array missing `IntoIterator` msg) - rust-lang#82917 (Add function core::iter::zip) - rust-lang#82993 (rustdoc: Use diagnostics for error when including sources) - rust-lang#83522 (Improve fs error open_from unix) - rust-lang#83548 (Always preserve `None`-delimited groups in a captured `TokenStream`) - rust-lang#83555 (Add #[inline] to io::Error methods) Failed merges: - rust-lang#83130 (escape_ascii take 2) r? `@ghost` `@rustbot` modify labels: rollup
5 participants
fixes #82602
r? @estebank do you know whether we can use the expr span in
rustc_on_unimplemented? The label isn't too great rn