[riot-notifications] [RIOT-OS/RIOT] drivers/include: add header definition for wdt (#11527)

Kaspar Schleiser notifications at github.com
Fri Jun 21 11:24:52 CEST 2019


> So save maybe two instructions in ROM?
> I'm all for being thrifty in regards to space, but I feel like this is pushing it a bit.

We get this argument *all the time*, like on hundreds of occasions, "why save those two bytes"?

Each individual two bytes (in this case it would be at least 4 on ARM) really do not matter. But all the hundreds of times we decide to not care do add up. In the end, there's kilobytes of "n<$much bytes do not matter" in the final binary.

Of course this kind of optimization should not go too far. But IMO a centralized ifdef guard is acceptable.

-- 
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/11527#issuecomment-504358299
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190621/c591243a/attachment.html>


More information about the notifications mailing list