[riot-notifications] [RIOT-OS/RIOT] sys/net/application_layer/nanocoap: Add path prefix option (#11098)

Ken Bannister notifications at github.com
Mon Mar 11 18:12:05 CET 2019


kb2ma commented on this pull request.



> @@ -323,7 +323,15 @@ ssize_t coap_handle_req(coap_pkt_t *pkt, uint8_t *resp_buf, unsigned resp_buf_le
             continue;
         }
 
-        int res = strcmp((char *)uri, resource->path);
+        int res;

As you know, I have been working to reduce duplication between nanocoap and gcoap. At the same time, the size of the API has grown with the addition of options. Build the source doc for #11057 and review the nanocoap page to see what I mean.

I stand by my previous comment. I don't see the value of this function to a user, but I'm OK with a private API to remove the duplication.

-- 
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/11098#discussion_r264337131
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190311/97a1bb18/attachment.html>


More information about the notifications mailing list