-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Have Error::Disconnected
contain the actor name
#146
Comments
I think that all the errors should probably contain actor name and message type where possible, too. This would mean having something like struct Error {
actor: Option<&'static str>,
message: Option<&'static str>,
kind: ErrorKind,
}
enum ErrorKind {
Interrupted,
Disconnected,
}
|
Related: is it worth making |
A message channel knows which actor it is targeting so I think this should still be possible? |
Ah, that is true! |
This will also be a breaking change so I think we should try and get it into 0.6. |
I am not sure I will have time to work on this any time soon. I don' think it is a must to get into What do you think? |
I agree! Can definitely postpone this. |
Perhaps we should at least make the struct opaque such that we can work on this feature without making a breaking change? |
Makes for better error messages.
The text was updated successfully, but these errors were encountered: