-
Notifications
You must be signed in to change notification settings - Fork 0
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
Draft: PPT mode #1
Conversation
…gn with wamp spec
|
||
@override | ||
bool Verify() { | ||
if (timeout! < 0) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Timeout possibly null at this point?
@@ -25,7 +25,7 @@ class Subscribed extends AbstractMessage { | |||
return _revokeCompleter.future; | |||
} | |||
|
|||
/// Is called by the session when the revokation happens | |||
/// Is called by the session when the revocation happens |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is what's currently breaking the build. revoke is calling compete with a nullable String, while the completer is a non-nullable string.
Main comment I have is that most things appear to be marked as nullable and there's liberal use of ! that assumes the data is not null. I haven't looked through the rest of the Connectanum code, so unsure if this is the norm here. |
Closing as this became obsolete: upstream made some updates and merged, our own fork also went ahead. |
This is a work in progress implementation of PPT Mode in lib for review and discussions