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

José Alamos notifications at github.com
Mon Jan 14 16:14:30 CET 2019


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

could you provide an example about how would you modify the example in order to make it clearer?

-- 
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/10622#discussion_r247529673
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190114/8f96ee80/attachment.html>


More information about the notifications mailing list