[riot-notifications] [RIOT-OS/RIOT] sys: new sock submodule for DTLS (#11909)

Aiman Ismail notifications at github.com
Wed Jul 31 10:47:43 CEST 2019


pokgak commented on this pull request.



> +#define MAIN_QUEUE_SIZE         (8)
+static msg_t _main_msg_queue[MAIN_QUEUE_SIZE];
+
+extern int dtls_client_cmd(int argc, char **argv);
+extern int dtls_server_cmd(int argc, char **argv);
+
+static const shell_command_t shell_commands[] = {
+    { "dtlsc", "Start a DTLS client", dtls_client_cmd },
+    { "dtlss", "Start and stop a DTLS server", dtls_server_cmd },
+    { NULL, NULL, NULL }
+};
+
+int main(void)
+{
+    /* we need a message queue for the thread running the shell in order to
+     * receive potentially fast incoming networking packets */

I copied this from the dtls-echo example but other examples such as gnrc_networking, cord_ep, gcoap (cord_ep and gcoap also uses `sock`) also have a similar comment. So maybe it is all copied and pasted from somewhere. Documentation about when the queue is needed somewhere would be nice.

Tested without the queue and it still works. I'll remove the queue.

-- 
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/11909#discussion_r309105177
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190731/0abb7e90/attachment.htm>


More information about the notifications mailing list