Broken Result type, missing Send/Sync in v0.7.1 about dipstick

[fralalonde]

Since Dipstick itself doesnt return errors and only propagates the errors from other layers, just boxing them up seemed to be the simplest way, no failure crate / dependency required. Could just adding Send + Sync to the Boxed trait satisfy your requirements? What would be the gain in returning a concrete Dipstick Error struct? All it would do (for now) is wrap other errors...

[vorner]

Send + Sync is a good start (things that are not are really hard to work with and mostly incompatible with any crate that helps with error handling). In the case of just propagating, it might be a good solution.

The advantage of an enum wrapper around concrete types are:

• You can see in documentation what erros may happen.
• All the Send, Sync and possibly other auto-traits are handled by the compiler.
• It's a bit easier to handle some of the errors in a different way than the rest, by matching. You can still downcast the boxed ones, but that's a bit more work.

[fralalonde]

I've made a PR to add Send + Sync to boxed errors.

For the record, I also I rewrote an Error enum, but pulled it back. Maintaining an Enum along with the required From impls is not something I want to commit to as it feels the whole Rust error management is still in shift anyway and a better way might yet come up. Also, it's not a showstopper, especially as the errors currently being reported are probably not rich enough to mandate handling them in a match. I understand your points though and will keep them in mind when it's time to revisit the error mechanism.