Skip to content
This repository has been archived by the owner on Aug 16, 2021. It is now read-only.

Regression with type inference in a recent nightly affects bail! #132

Closed
golddranks opened this issue Feb 20, 2017 · 5 comments
Closed

Regression with type inference in a recent nightly affects bail! #132

golddranks opened this issue Feb 20, 2017 · 5 comments

Comments

@golddranks
Copy link
Contributor

golddranks commented Feb 20, 2017 •
edited
Loading

One of the doc tests fails.

The breakage happens between nightlies 668864d9e 2017-02-16 and 536a900c4 2017-02-17.
@Yamakaky
Copy link
Contributor

Thanks!

For the record, the error is https://travis-ci.org/brson/error-chain/jobs/203564573#L385, with this line causing problem. Maybe it's because the Ok(()) is unreachable?

@Yamakaky Yamakaky mentioned this issue Feb 20, 2017
Closed
@Emilgardis
Copy link

I'm getting this also in recent nightly, but in code that previously worked.
Result<()> seems to be the problem.

@Yamakaky
Copy link
Contributor

Yamakaky commented Mar 5, 2017

What do you mean?

@Emilgardis
Copy link

I had an unreachable Ok(()), replacing it with unreachable! fixed it.

@Yamakaky
Copy link
Contributor

Yamakaky commented Mar 7, 2017

Yeah, it's an inference change from the compiler.

@Yamakaky Yamakaky closed this as completed Mar 7, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Yamakaky @Emilgardis @golddranks