On Tue, Jul 14, 2015 at 8:18 AM, Gurucharan Shetty <shet...@nicira.com> wrote: > On Mon, Jul 13, 2015 at 2:12 PM, Jesse Gross <je...@nicira.com> wrote: >> On Fri, Jul 10, 2015 at 9:26 PM, Gurucharan Shetty <shet...@nicira.com> >> wrote: >>> MSVC 2013 does not like designated initializers when >>> structs are initialized inside structs. >>> >>> Signed-off-by: Gurucharan Shetty <gshe...@nicira.com> >> >> This looks fine to me, although there are several other instances of >> this pattern in other places. Are the ones in ofproto/ofproto-dpif.c, >> for example, which also initialize .support different enough that they >> slip into some weird corner case that is OK? > I started looking more at the reason on why the change in dpif-netdev > causes failures and why the use of designated initializers in > ofproto-dpif does not. So one thing that I have in public domain is > this: > https://connect.microsoft.com/VisualStudio/feedback/details/805981/msvc-2013-c99-designated-initializers-cannot-initialize-unions-within-structs > > Based on the above, my first feeling is that because struct flow has > unions inside it, that may be the reason. I also noticed that > Microsoft had marked the issue as resolved. So I downloaded their > latest upgrade pack "visual studio 2103 update 4" and that does not > have any problems. Looks like Appveyor uses the same upgrade pack and > so we do not see the same problem at appveyor anymore. > > So I will just drop this patch and use the latest MSVC 2013 instead.
Awesome! _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
