[riot-notifications] [RIOT-OS/RIOT] ndp: provide define for common hop-limit (#11011)

Kaspar Schleiser notifications at github.com
Tue Feb 19 10:38:01 CET 2019


> Because it is adding dynamic code for static configuration (that moreover is not supposed to be changed).

I know what you mean, but we are talking about DEBUG statements, right? In that case, just imagine someone changing NDP_HOP_LIMIT just to still see "255" in the debug output. The WTF factor will be much higher than the couple of extra cycles needed to convert the number to a string. So I'd say not using the define is premature optimization.

-- 
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/11011#issuecomment-465058426
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190219/00c91b85/attachment.html>


More information about the notifications mailing list