[riot-notifications] [RIOT-OS/RIOT] SAUL: Reduce RAM requirements (#11392)

Sebastian Meiling notifications at github.com
Mon Apr 15 12:02:57 CEST 2019


I agree with the usage (and changes) of `fmt` the other changes I find a bit controversial. Mainly that adding a new sensor now requires to change 2 `enums` instead of one, however I like the general idea of having a class and start numbering from there.

TL;DR: please split this in 2 PRs one for `fmt` (which I would merge directly) and one for the other change which (to me) need some discussion, for instance @leandrolanzieri might have an opinion on this too.

-- 
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/11392#issuecomment-483189502
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190415/577a913d/attachment.html>


More information about the notifications mailing list