On Tue, Aug 01, 2017 at 10:24:47AM -0700, Joe Stringer wrote: > On 1 August 2017 at 07:55, Ben Pfaff <[email protected]> wrote: > > On Mon, Jul 31, 2017 at 05:16:11PM -0700, Joe Stringer wrote: > >> Clang complains: > >> > >> In file included from include/openvswitch/cxxtest.cc:11:0: > >> ../include/openvswitch/ofpbuf.h: In function ‘ofpbuf > >> ofpbuf_const_initializer(const void*, size_t)’: > >> ../include/openvswitch/ofpbuf.h:107:5: warning: narrowing conversion of > >> ‘size’ from ‘size_t {aka long unsigned int}’ to ‘uint32_t {aka unsigned > >> int}’ inside { } [-Wnarrowing] > >> }; > >> ^ > >> ../include/openvswitch/ofpbuf.h:107:5: warning: narrowing conversion of > >> ‘size’ from ‘size_t {aka long unsigned int}’ to ‘uint32_t {aka unsigned > >> int}’ inside { } [-Wnarrowing] > >> > >> This is because the ofpbuf struct's "size" parameter is a uint32_t, > >> while ofpbuf_const_initializer() takes a size_t for the size. Fix this > >> function to take a uint32_t instead. > >> > >> Signed-off-by: Joe Stringer <[email protected]> > > > > Thanks. (We must have different clang versions.0 > > > > Acked-by: Ben Pfaff <[email protected]> > > Perhaps - Clang-4.0? > > Thanks, applied to master.
Probably, I have clang-3.8 here. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
