search

contextfree / winrt-rust

Use and (eventually) make Windows Runtime APIs with Rust
Apache License 2.0
142 stars 10 forks source link

Implement std::error::Error for winrt::Error #69

Open frewsxcv opened 5 years ago

frewsxcv commented 5 years ago

Thoughts on implementing the primary Rust error trait for winrt's error type?

Boddlnagg commented 5 years ago

I think I haven't done that so far because std::error::Error has some deficiencies (which is why https://github.com/rust-lang-nursery/failure is a thing). I'm not sure if we are able to support both. Maybe with a feature flag?

frewsxcv commented 5 years ago

I think I haven't done that so far because std::error::Error has some deficiencies (which is why https://github.com/rust-lang-nursery/failure is a thing)

It has some deficiencies, yeah, but this RFC was approved which plans to improve it. This is your crate though, so up to you!