On Tue, Nov 05, 2019 at 02:57:46PM -0800, William Tu wrote:
> +    /* ODP_SUPPORT_FIELDS */
> +    str_value = xasprintf("%"PRIuSIZE, odp.max_vlan_headers);
> +    smap_add(cap, "max_vlan_headers", str_value);
> +    free(str_value);

I think that you can shorten the above to:
    smap_add_format(cap, "max_vlan_headers", "%"PRIuSIZE, odp.max_vlan_headers);
and similarly for other cases.

I think that we can improve the documentation.  I'm working on a
suggestion for that, please give me a while to write it up.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to