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.
V3 - fix examples as well Stephen Hemminger (7): 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 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 | 2 +- config/meson.build | 1 + drivers/meson.build | 2 +- 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 | 3 ++- 13 files changed, 27 insertions(+), 19 deletions(-) -- 2.47.2
