2 ms·
Yup. I absolutely would throw `#[from]` on everything when I started using thiserror, but now only do so in incredibly obvious cases like enum CarWontMove {
by skavi 2y ago
Yup. I absolutely would throw `#[from]` on everything when I started using thiserror, but now only do so in incredibly obvious cases like
enum CarWontMove {
EngineTroubles(EngineTroubles),
WheelsFellOff(WheelsFellOff),
}
Even then, there’s often some additional context you can affix at that higher level.
- shepmaster 2y agoSNAFU follows much the same idea: we have an attribute you can add [0] when you want to allow directly implementing `From`. Like thiserror, you can also mark an error as transparent [1] when even the error existing doesn't provide useful information. [0]: https://docs.rs/snafu/latest/snafu/derive.Snafu.html#disabling-the-context-selector https://docs.rs/snafu/latest/snafu/derive.Snafu.html#disabli... [1]: https://docs.rs/snafu/latest/snafu/derive.Snafu.html#delegating-to-the-underlying-error https://docs.rs/snafu/latest/snafu/derive.Snafu.html#delegat...