[riot-notifications] [RIOT-OS/RIOT] examples/cord_epsim: change default RD server addr handling (#10464)

Hauke Petersen notifications at github.com
Tue Mar 26 15:49:16 CET 2019

haukepetersen commented on this pull request.

> @@ -60,9 +66,14 @@ int cord_epsim_register(void)
     /* finish, we don't have any payload */
     ssize_t len = gcoap_finish(&pkt, 0, COAP_FORMAT_NONE);
-    if (gcoap_req_send2(buf, len, &remote, NULL) == 0) {
+    if (gcoap_req_send2(buf, len, rd_ep, _req_handler) == 0) {
         return CORD_EPSIM_ERROR;
     return CORD_EPSIM_OK;

Indeed I missed to set the `_state` to busy, that is fixed now. 

But for the return value, I still think returning `_OK` here makes sense, to signal that the registration request was sent our correctly.

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...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190326/d2adb5b8/attachment.html>

More information about the notifications mailing list