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

danpetry notifications at github.com
Thu Jan 10 14:01:29 CET 2019


danpetry commented on this pull request.



> +information.
+
+## 3.1. Registry handlers
+A RH represents a configuration group in the RIOT Registry. A RIOT
+module requires to implement and register a Registry Handler in order to expose
+its configurations to the Registry API.
+
+A RH is defined by a name and a series of handlers for
+interacting with the configuration parameters of the configuration group.
+These handlers are:
+- `set`: Sets a value to a configuration parameter. This handler MUST take care
+  of any logic to apply the value (e.g. data validation) or cache it until the
+commit handler is called
+- `get`: Gets the current value of a configuration parameter
+- `commit`: To be called when configuration parameters have been loaded from
+  storage. It could be use for special needed logic on applying configuration

Yep ok. What I meant was that saying

 "`commit`: To be called when configuration parameters have been loaded from storage"
Is a bit confusing, because `set` can also be called when configuration parameters have been loaded from storage. It would be less confusing to omit this sentence entirely here. Soemthing like:
`commit`: does the same as `set`, but also allows specially needed logic to be executed on applying configuration

-- 
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_r246749234
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190110/2490c993/attachment.html>


More information about the notifications mailing list