OK, I look forward to patches.
On Wed, Nov 20, 2013 at 07:07:23AM +0000, Alessandro Pilotti wrote: > +1 > > > > On 20.11.2013, at 00:17, "Ethan Jackson" <et...@nicira.com> wrote: > > > > Ben's the authority on this, but FWIW a macro sounds fine to me. > > We've already got PRI_U32 etc, adding a PRI_ZU seems reasonable. > > > > Ethan > > > > > > > > On Tue, Nov 19, 2013 at 1:57 PM, Alessandro Pilotti > > <apilo...@cloudbasesolutions.com> wrote: > >> Talking about %zu, we solved it with a brutal macro. Do you have any > >> suggestion for a better approach? > >> > >> > >> > >>> On 19 Nov 2013, at 18:51 , Gurucharan Shetty <shet...@nicira.com> wrote: > >>> > >>>> On Tue, Nov 19, 2013 at 8:37 AM, Ben Pfaff <b...@nicira.com> wrote: > >>>>> On Tue, Nov 19, 2013 at 12:22:40PM +0000, Alessandro Pilotti wrote: > >>>>> The freshly released Visual Studio 2013 is the minimum supported > >>>>> version due to its improved C99 support that avoided us a few > >>>>> headaches. > >>>> > >>>> Guru, have you been using the compiler from VS2013? I see that it still > >>>> does not implement %zu even 14 years after it was standardized, but I > >>>> guess that some of the other fixes should help us write good code, in > >>>> particular http://msdn.microsoft.com/en-us/library/vstudio/hh409293.aspx > >>>> says that VS2013: > >>>> > >>>> Supports these ISO C99 language features: > >>>> > >>>> _Bool > >>>> > >>>> Compound literals. > >>>> > >>>> Designated initializers. > >>>> > >>>> Mixing declarations with code. > >>>> > >>>> and the first three of those in particular are a breath of fresh air. > >>> I think the team here has been using VS2012. > >> > >> _______________________________________________ > >> dev mailing list > >> dev@openvswitch.org > >> http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
