Skip to content

Commit

Permalink
review comments: reword and add test
Browse files Browse the repository at this point in the history
  • Loading branch information
estebank committed Aug 21, 2019
1 parent ba8e094 commit a710c61
Show file tree
Hide file tree
Showing 3 changed files with 46 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/librustc/ty/error.rs
Original file line number Diff line number Diff line change
Expand Up @@ -269,8 +269,8 @@ impl<'tcx> TyCtxt<'tcx> {
let f_str = values.found.to_string();
if &e_str == &f_str && &e_str == "impl std::future::Future" {
// FIXME: use non-string based check.
db.help("if both futures resolve to the same type, consider `await`ing \
on both of them");
db.help("if both `Future`s have the same `Output` type, consider \
`.await`ing on both of them");
}
}
if let (ty::Infer(ty::IntVar(_)), ty::Float(_)) =
Expand Down
24 changes: 24 additions & 0 deletions src/test/ui/suggestions/opaque-type-error.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
// edition:2018
use core::future::Future;

async fn base_thing() -> Result<(), ()> {
Ok(())
}

fn thing_one() -> impl Future<Output = Result<(), ()>> {
base_thing()
}

fn thing_two() -> impl Future<Output = Result<(), ()>> {
base_thing()
}

async fn thing() -> Result<(), ()> {
if true {
thing_one()
} else {
thing_two() //~ ERROR if and else have incompatible types
}.await
}

fn main() {}
20 changes: 20 additions & 0 deletions src/test/ui/suggestions/opaque-type-error.stderr
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
error[E0308]: if and else have incompatible types
--> $DIR/opaque-type-error.rs:20:9
|
LL | / if true {
LL | | thing_one()
| | ----------- expected because of this
LL | | } else {
LL | | thing_two()
| | ^^^^^^^^^^^ expected opaque type, found a different opaque type
LL | | }.await
| |_____- if and else have incompatible types
|
= note: expected type `impl std::future::Future` (opaque type)
found type `impl std::future::Future` (opaque type)
= note: distinct uses of `impl Trait` result in different opaque types
= help: if both `Future`s have the same `Output` type, consider `.await`ing on both of them

error: aborting due to previous error

For more information about this error, try `rustc --explain E0308`.

0 comments on commit a710c61

Please sign in to comment.