This still needs a review.
On Thu, Jun 14, 2018 at 02:49:23PM -0700, Ben Pfaff wrote: > These don't trigger any new actual warnings in my own build. > > GCC 8.x adds other new warning options that are enabled by -Wall or > -Wextra. This commit doesn't explicitly enable those because OVS already > enables -Wall and -Wextra. > > Signed-off-by: Ben Pfaff <[email protected]> > --- > configure.ac | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/configure.ac b/configure.ac > index f0e4b51274d5..0bf8a78a9a56 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -170,6 +170,8 @@ OVS_ENABLE_OPTION([-Wshift-negative-value]) > OVS_ENABLE_OPTION([-Wduplicated-cond]) > OVS_ENABLE_OPTION([-Qunused-arguments]) > OVS_ENABLE_OPTION([-Wshadow]) > +OVS_ENABLE_OPTION([-Wmultistatement-macros]) > +OVS_ENABLE_OPTION([-Wcast-align=strict]) > 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 [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
