[dpdk-dev] [dpdk-dev,1/2] ethdev: remove useless null checks

2016-01-22 Thread Thomas Monjalon
2016-01-21 20:02, Jan Viktorin: > On Thu, 21 Jan 2016 12:57:10 +0100 > David Marchand wrote: > > - if ((name == NULL) || (pci_dev == NULL)) > > - return -EINVAL; > > Do you use a kind of assert in DPDK? The patch looks OK, however, I > would prefer something like > > assert_not

[dpdk-dev] [dpdk-dev,1/2] ethdev: remove useless null checks

2016-01-21 Thread Jan Viktorin
On Thu, 21 Jan 2016 12:57:10 +0100 David Marchand wrote: > We are in static functions and those passed arguments can't be NULL. > > Signed-off-by: David Marchand > > --- > lib/librte_ether/rte_ethdev.c | 15 --- > 1 file changed, 15 deletions(-) > > diff --git a/lib/librte_ether/r