Message ID | 20180709213755.13614-1-blp@ovn.org |
---|---|
State | Accepted |
Headers | show |
Series | [ovs-dev] configure: Disable -Wnull-pointer-arithmetic Clang warning. | expand |
Thanks. Reviewed-by: Yifeng Sun <pkusunyifeng@gmail.com> On Mon, Jul 9, 2018 at 2:37 PM, Ben Pfaff <blp@ovn.org> wrote: > OVS trips over this warning all over the place, so it's not worth leaving > on. > > Signed-off-by: Ben Pfaff <blp@ovn.org> > --- > configure.ac | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/configure.ac b/configure.ac > index af570b32012d..c89c607c7124 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -171,6 +171,7 @@ OVS_ENABLE_OPTION([-Wshift-negative-value]) > OVS_ENABLE_OPTION([-Wduplicated-cond]) > OVS_ENABLE_OPTION([-Qunused-arguments]) > OVS_ENABLE_OPTION([-Wshadow]) > +OVS_ENABLE_OPTION([-Wno-null-pointer-arithmetic]) > OVS_CONDITIONAL_CC_OPTION([-Wno-unused], [HAVE_WNO_UNUSED]) > OVS_CONDITIONAL_CC_OPTION([-Wno-unused-parameter], > [HAVE_WNO_UNUSED_PARAMETER]) > OVS_ENABLE_WERROR > -- > 2.16.1 > > _______________________________________________ > dev mailing list > dev@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev >
Thanks, applied to master. On Wed, Jul 11, 2018 at 01:25:03PM -0700, Yifeng Sun wrote: > Thanks. > > Reviewed-by: Yifeng Sun <pkusunyifeng@gmail.com> > > On Mon, Jul 9, 2018 at 2:37 PM, Ben Pfaff <blp@ovn.org> wrote: > > > OVS trips over this warning all over the place, so it's not worth leaving > > on. > > > > Signed-off-by: Ben Pfaff <blp@ovn.org> > > --- > > configure.ac | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/configure.ac b/configure.ac > > index af570b32012d..c89c607c7124 100644 > > --- a/configure.ac > > +++ b/configure.ac > > @@ -171,6 +171,7 @@ OVS_ENABLE_OPTION([-Wshift-negative-value]) > > OVS_ENABLE_OPTION([-Wduplicated-cond]) > > OVS_ENABLE_OPTION([-Qunused-arguments]) > > OVS_ENABLE_OPTION([-Wshadow]) > > +OVS_ENABLE_OPTION([-Wno-null-pointer-arithmetic]) > > OVS_CONDITIONAL_CC_OPTION([-Wno-unused], [HAVE_WNO_UNUSED]) > > OVS_CONDITIONAL_CC_OPTION([-Wno-unused-parameter], > > [HAVE_WNO_UNUSED_PARAMETER]) > > OVS_ENABLE_WERROR > > -- > > 2.16.1 > > > > _______________________________________________ > > dev mailing list > > dev@openvswitch.org > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > >
diff --git a/configure.ac b/configure.ac index af570b32012d..c89c607c7124 100644 --- a/configure.ac +++ b/configure.ac @@ -171,6 +171,7 @@ OVS_ENABLE_OPTION([-Wshift-negative-value]) OVS_ENABLE_OPTION([-Wduplicated-cond]) OVS_ENABLE_OPTION([-Qunused-arguments]) OVS_ENABLE_OPTION([-Wshadow]) +OVS_ENABLE_OPTION([-Wno-null-pointer-arithmetic]) OVS_CONDITIONAL_CC_OPTION([-Wno-unused], [HAVE_WNO_UNUSED]) OVS_CONDITIONAL_CC_OPTION([-Wno-unused-parameter], [HAVE_WNO_UNUSED_PARAMETER]) OVS_ENABLE_WERROR
OVS trips over this warning all over the place, so it's not worth leaving on. Signed-off-by: Ben Pfaff <blp@ovn.org> --- configure.ac | 1 + 1 file changed, 1 insertion(+)