[riot-notifications] [RIOT-OS/RIOT] sys: Add Link Format module (#11189)

Ken Bannister notifications at github.com
Thu Sep 26 12:27:08 CEST 2019


kb2ma commented on this pull request.



> +    CLIF_ATTR_TITLE         = 4,  /**< title */
+    CLIF_ATTR_TITLE_EXT     = 5,  /**< title* */
+    CLIF_ATTR_TYPE          = 6,  /**< type */
+    CLIF_ATTR_RT            = 7,  /**< rt */
+    CLIF_ATTR_IF            = 8,  /**< if */
+    CLIF_ATTR_SZ            = 9,  /**< sz */
+    CLIF_ATTR_CT            = 10, /**< ct */
+    CLIF_ATTR_OBS           = 11, /**< obs */
+    CLIF_ATTR_EXT           = 12  /**< extensions */
+} clif_attr_type_t;
+
+/**
+ * @brief Link format attribute descriptor
+ */
+typedef struct {
+    const char *value;             /**< string with the value */

I think the const qualifier here is too strong.

-- 
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/11189#pullrequestreview-293629802
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190926/621bdaa2/attachment.htm>


More information about the notifications mailing list