[riot-notifications] [RIOT-OS/RIOT] sys: Add RIOT Registry implementation (#10799)

danpetry notifications at github.com
Wed Feb 20 15:01:29 CET 2019

danpetry commented on this pull request.

> +     * failure
+     *
+     * @note The strings passed in @p argv do not contain the string of the name
+     * of the configuration group. E.g. If a parameter name is 'group/foo/var'
+     * and the name of the group is 'group', argv will contain 'foo' and 'var'.
+     */
+    char *(*hndlr_get)(int argc, char **argv, char *val, int val_len_max,
+                       void *context);
+    /**
+     * @brief Handler to set a the value of a configuration parameter.
+     * The value will be passed as a string in @p val.
+     *
+     * @param[in] argc Number of elements in @p argv
+     * @param[in] argv Parsed string representing the configuration parameter.
+     * @param[out] val Buffer containing the string of the new value

yep. so it would be a good idea to add a `val_len` parameter IMO

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/20190220/b6e58be6/attachment-0001.html>

More information about the notifications mailing list