[riot-notifications] [RIOT-OS/RIOT] WIP net/tcp: make header option pointer c++ compliant (#11300)

Martine Lenders notifications at github.com
Thu Mar 28 09:17:44 CET 2019


miri64 commented on this pull request.



>  } tcp_hdr_opt_t;
 
+/**
+ * @brief   Get the start of the TCP option value
+ *
+ * @param[in]   opt   TCP option field helper structure contiguous memory
+ *
+ * @returns     pointer to first byte of the value
+ */
+inline static uint8_t *tcp_hdr_opt_value_ptr(tcp_hdr_opt_t *opt){

Making inline functions non-static will cause the linker to fail once you include `net/tcp.h` in multiple c-files ;-). 

-- 
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/11300#discussion_r269893209
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190328/ecf00bae/attachment.html>


More information about the notifications mailing list