[riot-notifications] [RIOT-OS/RIOT] gnrc_rpl: do not assert netif on auto-init (#10895)

Martine Lenders notifications at github.com
Tue Jan 29 22:30:20 CET 2019


> My ceterum censeo is that we should not bloat the code with run-time checks, but rather get the compile-time dependencies straight. Adding a run-time check here is a very simple, but palliative solution. This check is unnecessary for correctly compiled applications, so that we add superfluous overhead for properly configured apps (which should be the majority of all compilations) ...

The problem is as e.g. in #10412 that this error was also caused by runtime errors (e.g. the radio failed to initialize for whatever reason). Basically, this assert (again: the *routing protocol* making assumptions about the lower layer) hardstops any attempt the lower layer might try to resolve the situation. IMHO the assert is completely in the wrong place.

-- 
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/10895#issuecomment-458714681
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190129/68d6aa07/attachment-0001.html>


More information about the notifications mailing list