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