Implement the Into<Option> trick for ApiError

ruma / homeserver

A Matrix homeserver written in Rust.

https://www.ruma.io/

1.08k stars 41 forks source link

Closed jimmycuadra closed 7 years ago

jimmycuadra commented 7 years ago

We should use the Into for the methods on ApiError that take an optional message. Then we don't have to wrap the messages in Some at the call site.

anuragsoni commented 7 years ago

If this is still open to pick, I'd like to try this one.

jimmycuadra commented 7 years ago

Sure thing. No one's opened a PR for it yet.

anuragsoni commented 7 years ago

@jimmycuadra I have made a pull request with an attempt

jimmycuadra commented 7 years ago

Implemented in #120.