Skip to content

Commit

Permalink
Make the "detect-old-time" UI test more representative
Browse files Browse the repository at this point in the history
The test code did have an inference failure, but that would have failed
on Rust 1.79 and earlier too. Now it is rewritten to be specifically
affected by 1.80's `impl FromIterator<_> for Box<str>`.
  • Loading branch information
cuviper committed Aug 29, 2024
1 parent 784d444 commit c339541
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 8 deletions.
Original file line number Diff line number Diff line change
@@ -1,8 +1,13 @@
#![crate_name = "time"]
#![crate_type = "lib"]

fn main() {
let items = Box::new(vec![]); //~ ERROR E0282
// This code compiled without error in Rust 1.79, but started failing in 1.80
// after the addition of several `impl FromIterator<_> for Box<str>`.

pub fn parse() -> Option<Vec<()>> {
let iter = std::iter::once(Some(())).map(|o| o.map(Into::into));
let items = iter.collect::<Option<Box<_>>>()?; //~ ERROR E0282
//~^ NOTE this is an inference error on crate `time` caused by an API change in Rust 1.80.0; update `time` to version `>=0.3.35`
Some(items.into())
//~^ NOTE type must be known at this point
//~| NOTE this is an inference error on crate `time` caused by an API change in Rust 1.80.0; update `time` to version `>=0.3.35`
items.into();
}
Original file line number Diff line number Diff line change
@@ -1,8 +1,11 @@
error[E0282]: type annotations needed for `Box<Vec<_>>`
--> $DIR/detect-old-time-version-format_description-parse.rs:4:9
error[E0282]: type annotations needed for `Box<_>`
--> $DIR/detect-old-time-version-format_description-parse.rs:9:9
|
LL | let items = Box::new(vec![]);
| ^^^^^ ---------------- type must be known at this point
LL | let items = iter.collect::<Option<Box<_>>>()?;
| ^^^^^
LL |
LL | Some(items.into())
| ---- type must be known at this point
|
= note: this is an inference error on crate `time` caused by an API change in Rust 1.80.0; update `time` to version `>=0.3.35` by calling `cargo update`

Expand Down

0 comments on commit c339541

Please sign in to comment.