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. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
