Re: [ovs-dev] [PATCH] types: Fix defined but not used warning.

2016-02-22 Thread Ben Pfaff
On Mon, Feb 22, 2016 at 11:19:07AM +0200, Panu Matilainen wrote: > On 02/19/2016 11:35 PM, William Tu wrote: > >warning: ‘OVS_BE128_MAX’ defined but not used [-Wunused-const-variable] > >Found using CentOS 6.6 with gcc 6.6.0. > > > >Signed-off-by: William Tu > > Just FWIW, this particular gcc 6 b

Re: [ovs-dev] [PATCH] types: Fix defined but not used warning.

2016-02-22 Thread Panu Matilainen
On 02/19/2016 11:35 PM, William Tu wrote: warning: ‘OVS_BE128_MAX’ defined but not used [-Wunused-const-variable] Found using CentOS 6.6 with gcc 6.6.0. Signed-off-by: William Tu Just FWIW, this particular gcc 6 behavior is still under debate: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=2890

Re: [ovs-dev] [PATCH] types: Fix defined but not used warning.

2016-02-19 Thread Ben Pfaff
OK, thanks. I adjusted that and applied this to master and branch-2.5. On Fri, Feb 19, 2016 at 02:38:02PM -0800, William Tu wrote: > Sorry it's 6.0.0 > CentOS 6.6 comes with gcc 4.4.7. I updated to version 6.0.0. > > # gcc --version > gcc (GCC) 6.0.0 20160217 (experimental) > Copyright (C) 2016

Re: [ovs-dev] [PATCH] types: Fix defined but not used warning.

2016-02-19 Thread William Tu
Sorry it's 6.0.0 CentOS 6.6 comes with gcc 4.4.7. I updated to version 6.0.0. # gcc --version gcc (GCC) 6.0.0 20160217 (experimental) Copyright (C) 2016 Free Software Foundation, Inc. On Fri, Feb 19, 2016 at 1:48 PM, Ben Pfaff wrote: > On Fri, Feb 19, 2016 at 01:35:55PM -0800, William Tu wrote

Re: [ovs-dev] [PATCH] types: Fix defined but not used warning.

2016-02-19 Thread Ben Pfaff
On Fri, Feb 19, 2016 at 01:35:55PM -0800, William Tu wrote: > warning: ‘OVS_BE128_MAX’ defined but not used [-Wunused-const-variable] > Found using CentOS 6.6 with gcc 6.6.0. Are you sure about that GCC version number? ___ dev mailing list dev@openvswitc

[ovs-dev] [PATCH] types: Fix defined but not used warning.

2016-02-19 Thread William Tu
warning: ‘OVS_BE128_MAX’ defined but not used [-Wunused-const-variable] Found using CentOS 6.6 with gcc 6.6.0. Signed-off-by: William Tu --- include/openvswitch/types.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/include/openvswitch/types.h b/include/openvswitch/types.h