[PATCH v2 3/6] graph: replace unnecessary comma operator

Bruce Richardson bruce.richardson at intel.com
Mon Aug 18 10:01:23 CEST 2025


On Thu, Aug 14, 2025 at 02:28:49PM -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/graph/graph_stats.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/graph/graph_stats.c b/lib/graph/graph_stats.c
> index eac73cbf71..588508b473 100644
> --- a/lib/graph/graph_stats.c
> +++ b/lib/graph/graph_stats.c
> @@ -254,7 +254,7 @@ stats_mem_populate(struct rte_graph_cluster_stats **stats_in,
>  	*stats_in = NULL;
>  
>  	/* Clear the new struct cluster_node area */
> -	cluster = RTE_PTR_ADD(stats, stats->sz),
> +	cluster = RTE_PTR_ADD(stats, stats->sz);
>  	memset(cluster, 0, stats->cluster_node_size);
>  	memcpy(cluster->stat.name, graph_node->node->name, RTE_NODE_NAMESIZE);
>  	cluster->stat.id = graph_node->node->id;

Acked-by: Bruce Richardson <bruce.richardson at intel.com>



More information about the dev mailing list