programatik29 / tokio-rusqlite

Asynchronous handle for rusqlite library.
MIT License
89 stars 21 forks source link

Add support for application specific errors #19

Closed czocher closed 1 year ago

czocher commented 1 year ago

Closes #18

programatik29 commented 1 year ago

As we have Other variant, should the error enum be #[non_exhaustive] anymore?

czocher commented 1 year ago

#[non_exhaustive] gives us the ability to specify other errors in the future, so it should remain for backwards-compatibility in my opinion. It's overall a suggested approach towards error enums in Rust.