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