Noticed in ethdev an obvious case of extra parenthesis so went hunting for others as well. There seems to be some weird coding standard that loves extra parenthesis (or developers don't understand basic C precedence rules).
Stephen Hemminger (5): bnx2x: remove (redundant) parenthesis kni: remove (redundant) parenthesis ethdev: remove (redundant) parenthesis distributor: remove (unnecessary) parenthesis loadbalancer: remove (unnecessary) parenthesis drivers/net/bnx2x/bnx2x.c | 2 +- drivers/net/bnx2x/ecore_sp.c | 4 ++-- examples/load_balancer/config.c | 2 +- lib/librte_distributor/rte_distributor.c | 2 +- lib/librte_eal/linuxapp/kni/ethtool/igb/igb_main.c | 4 ++-- lib/librte_eal/linuxapp/kni/ethtool/ixgbe/ixgbe_common.c | 6 +++--- lib/librte_ether/rte_ethdev.c | 4 ++-- 7 files changed, 12 insertions(+), 12 deletions(-) -- 2.11.0
