[PATCH v2 2/6] ipsec: replace unnecessary comma operator

Bruce Richardson bruce.richardson at intel.com
Mon Aug 18 10:00:52 CEST 2025


On Thu, Aug 14, 2025 at 02:28:48PM -0700, Stephen Hemminger wrote:
> Use of comma as statement separator is discouraged and
> reported as warning by clang with -Wcomma
> 
> Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>
> ---
>  lib/ipsec/misc.h | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/ipsec/misc.h b/lib/ipsec/misc.h
> index fd4d3f5256..54dc5c2d4d 100644
> --- a/lib/ipsec/misc.h
> +++ b/lib/ipsec/misc.h
> @@ -126,7 +126,8 @@ cpu_crypto_bulk(const struct rte_ipsec_session *ss,
>  
>  	const uint32_t vnum = RTE_DIM(vec);
>  
> -	j = 0, n = 0;
> +	j = 0;
> +	n = 0;
>  	vofs = 0;
>  	for (i = 0; i != num; i++) {
>  

A better fix here is to move the assignments up to where the variables are
defined just 9 lines earlier, rather than wasting code lines by assigning
to zero later on!


More information about the dev mailing list