-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Use enum class to define protocols message type for strong typing #4593
Use enum class to define protocols message type for strong typing #4593
Conversation
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.
Thank you!
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.
Thank you!
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.
Approving given @pan-apple 's approval
Review comments addressed
…oject-chip#4593) * Use enum class to define protocols message type for strong typing * Apply suggestions from code review Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
…oject-chip#4593) * Use enum class to define protocols message type for strong typing * Apply suggestions from code review Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
…oject-chip#4593) * Use enum class to define protocols message type for strong typing * Apply suggestions from code review Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
Problem
This is follow-up of #4429 which defined a set of APIs that allow us to just pass in a strongly typed (enum class) message type and derive the protocol from the type of the message type. Currently, only SecureChannel protocol has been updated to use strongly typed (enum class) message type.
Summary of Changes
update all protocol message type to use strongly typed (enum class) message type.
Fixes #4592