Make DBM load methods return Option<T> instead of Result<T, E>

talaia-labs / rust-teos

The Eye of Satoshi - Lightning Watchtower

https://talaia-labs.github.io/talaia.watch/

MIT License

135 stars 63 forks source link

Make DBM load methods return Option<T> instead of Result<T, E> #54

Closed sr-gi closed 1 year ago

sr-gi commented 2 years ago

Currently the DBM methods related to loading data return Result<T, E> where E is always dbm::Error::NotFound. It may make more sense to get rit of NotFound from the Errors enum and make the load functions return Option<T>.

sr-gi commented 2 years ago

@SpoonBuoy just realized this may be relevant to your current work.