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

danpetry notifications at github.com
Tue Jan 15 11:48:47 CET 2019

danpetry 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)

Sorry, having a second look at it I reckon that the space that's been added makes it a lot more clear

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/20190115/2035b03f/attachment.html>

More information about the notifications mailing list