[riot-notifications] [RIOT-OS/RIOT] gnrc/netif: add gnrc_netif_ipv6_wait_for_prefix() (#16824)

benpicco notifications at github.com
Tue Sep 7 18:01:50 CEST 2021


@benpicco commented on this pull request.



> +#ifndef CONFIG_NETIF_NUM_MAX
+#define CONFIG_NETIF_NUM_MAX 2
+#endif

> > I could also use VLA instead fearful
> 
> https://en.wikipedia.org/wiki/Vla?

yummy, but I meant 

https://en.wikipedia.org/wiki/Variable-length_array#C99

They should generally be avoided because of their unpredictable nature and no way to check if stack space was exhausted.
However, this might actually be a valid use case: `msg_bus_entry_t` is only 12 byte and the number of netifs is also small, so it's unlikely that this will overflow the stack.


-- 
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/16824#discussion_r703642147
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210907/f9ebe3d1/attachment.htm>


More information about the notifications mailing list