-
-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
Minor Zephyr port features #11360
Minor Zephyr port features #11360
Conversation
880888f
to
26bfc5e
Compare
ports/zephyr/modusocket.c
Outdated
} | ||
|
||
if (recv_len == 0) { | ||
return mp_const_empty_bytes; |
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.
I think in this case it should still return a tuple, like (b"", <addr>)
.
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.
I made this changed during merge.
Thanks for the contribution! Apart from the above comment, these changes look good. |
This is an automated heads-up that we've just merged a Pull Request See #13763 A search suggests this PR might apply the STATIC macro to some C code. If it Although this is an automated message, feel free to @-reply to me directly if |
Signed-off-by: Jason Kridner <jkridner@beagleboard.org>
Signed-off-by: Jason Kridner <jkridner@beagleboard.org>
26bfc5e
to
c211c05
Compare
recvfrom
function useful for broadcast UDP messagesmdns_service: add ability to broadcast mdns advertisements