Myujth

I'm trying to implement an error enum which can contain an error associated with one of our traits like this:

trait Storage 
 type Error;


enum MyError<S: Storage> 
 StorageProblem(S::Error),

I have also tried to implement the From trait to allow construction of MyError from an instance of a Storage::Error:

impl<S: Storage> From<S::Error> for MyError<S> 
 fn from(error: S::Error) -> MyError<S> 
 MyError::StorageProblem(error)
 

(playground)

However this fails to compile:

error[E0119]: conflicting implementations of trait `std::convert::From<MyError<_>>` for type `MyError<_>`:
 --> src/main.rs:9:1
 |
9 | / impl<S: Storage> From<S::Error> for MyError<S> 
 | |_^
 |
 = note: conflicting implementation in crate `core`

I don't understand why the compiler reckons this has already been implemented. The error message is telling me that there's already an implementation of From<MyError<_>> (which there is), but I'm not trying to implement that here - I'm trying to implement From<S::Error> and MyError is not the same type as S::Error from what I can see.

Am I missing something fundamental to generics here?

The problem here is someone may implement Storage so that the From impl you have written overlaps with the impl in the standard library of impl<T> From<T> for T (that is, anything can be converted to itself).

Specifically,

struct Tricky;

impl Storage for Tricky 
 type Error = MyError<Tricky>;

(The set-up here means this doesn't actually compile—MyError<Tricky> is infinitely large—but that error is unrelated to the reasoning about impls/coherence/overlap, and indeed small changes to MyError can make it compile without changing the fundamental problem, e.g. adding a Box like StorageProblem(Box<S::Error>),.)

If we substitute Tricky in place of S in your impl, we get:

impl From<MyError<Tricky>> for MyError<Tricky> 
 ...

This impl exactly matches the self-conversion one with T == MyError<Tricky>, and hence the compiler wouldn't know which one to choose. Instead of making an arbitrary/random choice, the Rust compiler avoids situations like this, and thus the original code must be rejected due to this risk.

This coherence restriction can definitely be annoying, and is one of reasons that specialisation is a much-anticipated feature: essentially allows manually instructing the compiler how to handle overlap... at least, one of the extensions to the current restricted form allows that.

A workaround for the coherence issue is to use Result::map_err to perform the conversion yourself. You can then use the end Result with try! or ?:

fn example<S: Storage>(s: S) -> Result<i32, MyError<S>> 
 s.do_a_thing().map_err(MyError::StorageProblem)?;
 Ok(42)

This solution is also valuable when there are error variants that have the same underlying Error, such as if you want to separate "file opening" and "file reading" errors, both of which are io::Error.