[riot-notifications] [RIOT-OS/RIOT] net/sock: Add access to auxiliary data (API only) (#14703)

chrysn notifications at github.com
Wed Nov 4 11:51:34 CET 2020


@chrysn commented on this pull request.



> @@ -409,8 +465,47 @@ int sock_udp_get_remote(sock_udp_t *sock, sock_udp_ep_t *ep);
  *          the remote of @p sock.
  * @return  -ETIMEDOUT, if @p timeout expired.
  */
-ssize_t sock_udp_recv(sock_udp_t *sock, void *data, size_t max_len,
-                      uint32_t timeout, sock_udp_ep_t *remote);
+ssize_t sock_udp_recv_aux(sock_udp_t *sock, void *data, size_t max_len,

Then duplication is fine with me here (maybe I'll find the spoons to pluck that crow with doxygen); a note on equivalence a la

```
This is functionally equivalent with calling @ref sock_dtls_recv_aux with `NULL` as the @p aux parameter.
```

and

```
When @p aux is not needed, @ref sock_dtls_recv is provided as a shortcut with otherwise identical parameters.
```

might make the navigation easier to navigate.

(On the general review level, I'm still waiting for a more qualified fundamentals review.)

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/14703#discussion_r517257149
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20201104/f59a8be0/attachment.htm>


More information about the notifications mailing list