Re: [lwip-devel] API change: restructuring ipv4/ipv6 integration (task #12722)

[Simon Goldschmidt]

Sylvain Rochet wrote:
> Unfortunately, we are now hitting GCC bug #53119 [1][2][3].
>
> lwip/src/core/ipv4/ip4_addr.c:53:3: warning: missing braces around 
> initializer [-Wmissing-braces]
> }, IPADDR_TYPE_V4
> ^
> lwip/src/core/ipv4/ip4_addr.c:53:3: warning: (near initialization for 
> ‘ip_addr_any.addr.ip4’) [-Wmissing-braces]
> [..]
> Adding an extra brace level shut up the compiler warning, does it still
> work for all compilers ?

I think it should. Seems like I just got the initializer wrong and MSVC doesn't 
care :-(
Anyway, that GCC bug seems to be a bit different in that the generic short-zero 
initializer '{0}' doesn't work. But except for 'ip_addr_any', we initialize to 
non-zero, so that's something different.

Reading about this makes me think the order of ip4/ip6 in the ip_addr_t union 
needs to be changed to allow constant initialization of the larger ip6 part...


Simon