[dpdk-dev] [PATCH] enic: fix warnings

2014-12-04 Thread Thomas Monjalon
> >A lot of warnings were not seen because $(WERROR_FLAGS) was not set > >in the Makefile. But they appear with toolchains that enforce more checks. > > > >-Wno-deprecated seems useless. > >-Wno-strict-aliasing is added to avoid false positives. > > > >This patch cleans up unused variable, unused f

[dpdk-dev] [PATCH] enic: fix warnings

2014-12-04 Thread Thomas Monjalon
2014-12-04 10:29, Sujith Sankar: > > On 02/12/14 8:29 pm, "Thomas Monjalon" wrote: > > >A lot of warnings were not seen because $(WERROR_FLAGS) was not set > >in the Makefile. But they appear with toolchains that enforce more checks. > > > >-Wno-deprecated seems useless. > >-Wno-strict-aliasing

[dpdk-dev] [PATCH] enic: fix warnings

2014-12-04 Thread Sujith Sankar (ssujith)
On 04/12/14 4:18 pm, "Thomas Monjalon" wrote: >2014-12-04 10:29, Sujith Sankar: >> >> On 02/12/14 8:29 pm, "Thomas Monjalon" >>wrote: >> >> >A lot of warnings were not seen because $(WERROR_FLAGS) was not set >> >in the Makefile. But they appear with toolchains that enforce more >>checks. >>

[dpdk-dev] [PATCH] enic: fix warnings

2014-12-04 Thread Sujith Sankar (ssujith)
On 02/12/14 8:29 pm, "Thomas Monjalon" wrote: >A lot of warnings were not seen because $(WERROR_FLAGS) was not set >in the Makefile. But they appear with toolchains that enforce more checks. > >-Wno-deprecated seems useless. >-Wno-strict-aliasing is added to avoid false positives. > >This patch

[dpdk-dev] [PATCH] enic: fix warnings

2014-12-03 Thread Thomas Monjalon
2014-12-03 14:52, Sujith Sankar: > > On 02/12/14 8:29 pm, "Thomas Monjalon" wrote: > > >A lot of warnings were not seen because $(WERROR_FLAGS) was not set > >in the Makefile. But they appear with toolchains that enforce more checks. > > > >-Wno-deprecated seems useless. > >-Wno-strict-aliasing

[dpdk-dev] [PATCH] enic: fix warnings

2014-12-03 Thread Sujith Sankar (ssujith)
On 03/12/14 8:35 pm, "Thomas Monjalon" wrote: >2014-12-03 14:52, Sujith Sankar: >> >> On 02/12/14 8:29 pm, "Thomas Monjalon" >>wrote: >> >> >A lot of warnings were not seen because $(WERROR_FLAGS) was not set >> >in the Makefile. But they appear with toolchains that enforce more >>checks. >>

[dpdk-dev] [PATCH] enic: fix warnings

2014-12-03 Thread Sujith Sankar (ssujith)
On 02/12/14 8:29 pm, "Thomas Monjalon" wrote: >A lot of warnings were not seen because $(WERROR_FLAGS) was not set >in the Makefile. But they appear with toolchains that enforce more checks. > >-Wno-deprecated seems useless. >-Wno-strict-aliasing is added to avoid false positives. > >This patch

[dpdk-dev] [PATCH] enic: fix warnings

2014-12-02 Thread Thomas Monjalon
A lot of warnings were not seen because $(WERROR_FLAGS) was not set in the Makefile. But they appear with toolchains that enforce more checks. -Wno-deprecated seems useless. -Wno-strict-aliasing is added to avoid false positives. This patch cleans up unused variable, unused functions, wrong types