[riot-notifications] [RIOT-OS/RIOT] sys/posix: add module posix_netdb (#16853)
notifications at github.com
Tue Sep 14 15:57:20 CEST 2021
@miri64 requested changes on this pull request.
> I also added the option to use posix sockets for the existing `sock_dns` module. Without using posix sockets I had extremely poor success rates for address resolution (approximately 1 out of 20).
NACK. 1. We rather should find the reason, why `sock_udp` performs so poorly compared to POSIX sockets (which below take use `sock_udp`, so this fact seems weird to me). 2. `sock_dns` using POSIX sockets turns everything on its head, now we have a `sock` module using POSIX sockets using a `sock` module. Conceptually alone this sounds very dirty.
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications