Right now, we only support an underlying error denoted by the field named <code class=

Support additional ways of specifying the error source or backtrace about snafu HOT 3 CLOSED

shepmaster commented on August 16, 2024

Support additional ways of specifying the error source or backtrace

from snafu.

Comments (3)

lnicola commented on August 16, 2024

Should the inner error be optional? Right now you get a compile error if you don't have one.

from snafu.

shepmaster commented on August 16, 2024

Should the inner error be optional

I’m not sure I’m following. If you don’t include a source field, then there will be no inner error.

Do you mean optional as in source: Option<SomeError>? If so, could you expand a bit about why you’d want that and what the use case is?

from snafu.

lnicola commented on August 16, 2024

In your words :-):

If the original variant had a source field, its context selector will have an implementation of From for a Context

I didn't notice that, and this error got me a little confused, because it's not exactly clear what the issue is:

error[E0277]: the trait bound `Error: std::convert::From<snafu::Context<std::io::Error, Rename<&std::path::Path>>>` is not satisfied
   --> src/main.rs:113:13
    |
113 |             fs::rename(path, cleaned).context(Rename { path })?;
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `std::convert::From<snafu::Context<std::io::Error, Rename<&std::path::Path>>>` is not implemented for `Error`
    |
    = help: the following implementations were found:
              <Error as std::convert::From<snafu::Context<snafu::NoneError, PathDecode<__T0, __T1>>>>
              <Error as std::convert::From<snafu::Context<snafu::NoneError, Rename<__T0>>>>
              <Error as std::convert::From<snafu::Context<snafu::NoneError, UnsupportedEncoding<__T0>>>>
              <Error as std::convert::From<snafu::Context<uchardet::errors::Error, UnknownEncoding<__T0>>>>
              <Error as std::convert::From<snafu::Context<walkdir::Error, Enumerate>>>
    = note: required by `std::convert::From::from`

I'm not saying you should make that code work, it's just a thing I ran into.

from snafu.

Recommend Projects