Just as a clarification, removing the block of code causes it to compile correctly.
However, It doesnt really bode well for this code having been tested...
Edit: no it doesnt.
We use LWIP but without wiced, only with WWD. We have a rule about compiling without warnings (with a static analyzer) and so far we have made about 100 changes just to make lwip+wwd compile to our standards. Most stuff is mixing return types and such, but there have been a few questionable lines of code.
My recommendation is to compile as strictly as possible, and to put #ifdefs around things you notice, to fix them, and not to be shy about fixing things that are just plain wrong.
We've been doing this since SDK-2.3 and migrating forward is less painful every time since we usually merge in our modifications into the new releases.
EDIT: I also brought in LWIP 1.4.1 which is an official release, to replace LWIP 1.4.0.rc1, because why would we use an unofficial released version instead of the latest official release?
felix_turgeon I also brought in 1.4.1. My goal was to compile one of the demo apps that shipped with ThreadX support with LwIP instead because ThreadX errors are undebuggable. So I need the WICED apis for the moment.
EDIT: i wonder if theres a process for submitting fixes back to broadcom...