Add support for `dyn Error` references

Question

Add support for `dyn Error` references

demurgos opened this issue a year ago · 2 comments

The PR #327 adds support for error references. Unfortunately it's missing support for dyn Error. Error trait objects are fairly common so it's unfortunate that it's not supported, we should try to add support for it.

Answer 1 · 2024-01-04T03:22:43.000Z

As far as I know, using an owned trait object (Box<dyn Error>) should work fine with the existing ErrorValue. The only problem is using a borrowed &dyn Error with the new ErrorRef.

Please let me know if this is incorrect.

Answer 2 · 2024-09-23T17:47:52.000Z

Ah, we need to change the bound to E: ?Sized + std::error::Error (and add a new test for this scenario).