[riot-notifications] [RIOT-OS/RIOT] doc/memos: RDM Runtime Configuration Architecture (#10622)

Leandro Lanzieri notifications at github.com
Mon Jan 14 14:47:39 CET 2019

leandrolanzieri commented on this pull request.

> +    if(is_enabled) {
+        trigger_something();
+    }
+    /* As stated before, the application crashes if `set_threashold` is called when is_enabled is false.
+    We protect it here */
+    if(is_enabled) {
+        /* We can safely set the theshold without crashing the app */
+        set_threshold(threshold);
+    }
+/* Dummy implementation of `export` handler.
+   This handler has to call `export_func` for one or all parameters.
+   There can be different behaviors depending on the export function (e.g printing all configs
+   to STDOUT, save them in a non-volatile storage device, etc)

>     /* Prepare `buf` to contain is_enabled in a string representation */
>     /* ... */

In that comment it is implied that somehow the value of the variable is represented as a string. I don't really think it is central for showing how the export handler can be implemented, the main idea is to show how the export_func should be called.

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/20190114/21281380/attachment.html>

More information about the notifications mailing list