[PATCH v4 0/9] avoid unnecessary use of commaa operator
Stephen Hemminger
stephen at networkplumber.org
Mon Aug 18 18:18:05 CEST 2025
Clang has ability to detect suspicious use of comma operator
where a semicolon would have the same effect.
Easy enough to fix in DPDK code.
v4 - review feedback about consolidating assignments
fix Broadcom drivers (Intel and Nvidia still have warnings)
Stephen Hemminger (9):
eal: replace unnecessary comma operator
ipsec: replace unnecessary comma operator
graph: replace unnecessary comma operator
test: replace unnecessary comma operators
app/testpmd: replace comma operator with bracket
examples: remove unnecessary use of comma operator
net/bnx2x: remove unnecessary comma operator
net/bnxt: remove unnecessary use of comma operator
config: enable comma warnings
app/test-pmd/cmdline_flow.c | 6 ++++--
app/test-pmd/config.c | 10 ++++++----
app/test/test_compressdev.c | 4 ++--
app/test/test_link_bonding_rssconf.c | 2 +-
app/test/test_thash.c | 7 +++----
config/meson.build | 1 +
drivers/meson.build | 2 +-
drivers/net/bnx2x/bnx2x.c | 4 ++--
drivers/net/bnxt/tf_core/tf_msg.c | 4 ++--
examples/l2fwd-event/l2fwd_event_generic.c | 2 +-
examples/l3fwd/l3fwd_event_generic.c | 2 +-
lib/eal/common/rte_malloc.c | 4 ++--
lib/eal/linux/eal_alarm.c | 6 ++++--
lib/graph/graph_stats.c | 2 +-
lib/ipsec/misc.h | 8 +++-----
15 files changed, 34 insertions(+), 30 deletions(-)
--
2.47.2
More information about the dev
mailing list