[riot-notifications] [RIOT-OS/RIOT] sys/entropy_source: add new module for entropy sources (#14324)
notifications at github.com
Wed Nov 25 12:15:33 CET 2020
@maribu commented on this pull request.
> + * Although APIs between multiple entropy sources might be similar, every source has to define its
+ * own API to be open for specific requirements. Entropy sources are crucial, thus, separate APIs should
+ * make the choice for one or the other explicit. Furthermore, multiple sources should be able to run in parallel
+ * and separate APIs simplify access (source management).
Still, this whole paragraph should be removed and the API should be marked as internal. That way, refactoring the implementation to a common API can be done without maintaining multiple APIs.
In fact, IMO we should also mark the common entropy source API(s) as internal. I don't think that any application developer should ever directly interact with entropy sources, but instead interface with an entropy collection and mixing subsystem. Documenting the low level API(s) as internal might be one additional motivation for application programmers to do the right thing.
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...
More information about the notifications