[riot-notifications] [RIOT] net/fib: added configuration header for the FIB (#3022)

BytesGalore notifications at github.com
Mon Jun 15 16:09:11 CEST 2015


@Lotterleben changed to configure on compile time, but this may produce unwanted side-effects.
The defines are set right for the FIB/Universal address only (bad news).  

When including the `ng_fib.h` in a main, for instance in the `hello-world` example (with `USEMODULE += fib` of course) and calling say `make UNIVERSAL_ADDRESS_SIZE=17`, 
the main will use the `UNIVERSAL_ADDRESS_SIZE` default value of **`16`**, while the FIB will initialize and use it with the new value **`17`**.  (same applies for all other defines)

You can reproduce it by e.g. printing the value of the defines in the FIB (e.g. in `fib_init()`) and in the main accordingly.  
So I recommend to not provide this compile time configuration. 

---
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/3022#issuecomment-112082379
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20150615/7d79324e/attachment.html>


More information about the notifications mailing list