[riot-notifications] [RIOT-OS/RIOT] drivers/pulse_counter: refactor the prototypes of read funcs (#10716)

Kees Bakker notifications at github.com
Mon Jan 7 20:25:46 CET 2019


@cladmi Thanks for the pointer to #7883

It's a bit late, but I don't agree with @haukepetersen . I want every drop of `const` to be documented with a comment. It should only be used in exceptional cases. Perhaps here we have one, perhaps not. The question is: why were the saul read/write designed this way with a `const` pointer? There must be a reason otherwise we better remove the `const` from the interface.

-- 
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/10716#issuecomment-452051552
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190107/870768e4/attachment.html>


More information about the notifications mailing list