On Wed, 23 Sep 2020 19:06:05 +0100
Cristian Dumitrescu <cristian.dumitre...@intel.com> wrote:

> +/*
> + * Pipeline.
> + */
> +struct rte_swx_pipeline {
> +     int build_done;
> +     int numa_node;
> +};
> +

Nit, could build_done be a bool type?

+void
+rte_swx_pipeline_free(struct rte_swx_pipeline *p)
+{
+       if (!p)
+               return;
+
+       free(p);
+}

The free() function in libc is defined to accept NULL as ok.
Please remove the if()

Reply via email to