error: allow backtrace Option to be Display on stable#258
error: allow backtrace Option to be Display on stable#258yoshuawuyts merged 1 commit intohttp-rs:mainfrom
Conversation
be505e0 to
b05488c
Compare
|
Why wouldn't the user just test if the option is None? |
|
Ohhh… I see what's going on. What if we just used some already available type like |
|
I don't understand? |
|
@Fishrock123 sorry, forgot that comments don't auto escape |
This prevents the need for users to do backtrace feature-checking for code that wishes to use `Display` for a backtrace and also compile on stable.
b05488c to
10c3f60
Compare
|
I would rather it be a type that implements exactly the same traits as |
|
Ultimately, the correct answer here is that it would be |
|
Yep, agreed that |
yoshuawuyts
left a comment
There was a problem hiding this comment.
Oh, this is a fun patch. Looking good!
This prevents the need for users to do backtrace feature-checking for code that wishes to use
Displayfor a backtrace and also compile on stable.