Thank you guys for figuring that out and for the fix! I’ll quickly send a v4
> On 27 Mar 2015, at 15:58, Ben Pfaff <b...@nicira.com> wrote: > > Thanks for checking. (That was fast!) > > Daniele, maybe you should use something like this, then: > > struct u64_stats_sync { > #ifndef __GNUC__ > int happy_compiler; > #endif > }; > > On Fri, Mar 27, 2015 at 08:55:54AM -0700, Gurucharan Shetty wrote: >> MSVC does not allow empty structures. Thank you for noticing this! >> >> On Fri, Mar 27, 2015 at 8:25 AM, Ben Pfaff <b...@nicira.com> wrote: >>> On Fri, Mar 27, 2015 at 01:26:04PM +0000, Daniele Di Proietto wrote: >>>> +struct u64_stats_sync { >>>> + >>>> +}; >>> >>> Guru, does MSVC accept empty structures? I don't think they're >>> acceptable by the words of the C standards, but GCC and Clang allow >>> them. >>> >>> Thanks, >>> >>> Ben. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev