Re: [lwip-users] Patch for IGMP V1/V2 and raw multicast support

[Christiaan Simons]

The contributed IGMP + Altera code won't work out of the box for the raw
API + NO_SYS case.

There are a few assumptions on having a sys_arch
( sys_timeout() in igmp.c and etharp.c ),
therefore it isn't entirely "in the spirit of LWIP".

Also I miss some return value checks on mem_malloc().
It would be nice if these issues are solved first.

>From my perspective "the spirit of LWIP" is:

* Raw-API first, additional layers and API's can be
nice but should not collide or invade in the basic API.
* Robust, portable code.

Bye,

Christiaan Simons

Hardware Designer
Axon Digital Design

http://www.axon.tv