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
test that we tolerate out-of-band data #1197
base: main
Are you sure you want to change the base?
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.
Looks good minus the prb failing!
public static func sendto(descriptor: CInt, | ||
pointer: UnsafeRawPointer, | ||
size: size_t, | ||
flags: CInt, |
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.
Isn’t this an API breaking change? Or am I missing something?
How come we have public methods on an internal enum?
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.
Fine modulo @glbrntt’s feedback.
713cf14
to
83b0ef4
Compare
New test failed on 5.1... 👎 |
Motivation: We don't support OOB data but we should make sure that nothing breaks if OOB arrives. Modifications: Add a test that makes sure we tolerate OOB data. Result: More test coverage.
83b0ef4
to
c22e552
Compare
Motivation:
We don't support OOB data but we should make sure that nothing breaks if
OOB arrives.
Modifications:
Add a test that makes sure we tolerate OOB data.
Result:
More test coverage.