Re: [ovs-dev] [PATCH 2/3] ofp-util: New data structure for mapping between table names and numbers.

2018-01-16 Thread Yifeng Sun
It looks like that the not-null-validation of 'map' can be removed in static function ofputil_name_map_destroy but should be added for ofputil_port_map_destroy and ofputil_table_map_destroy. Other than that, this patch looks good to me. Thanks. Reviewed-by: Yifeng Sun

[ovs-dev] [PATCH 2/3] ofp-util: New data structure for mapping between table names and numbers.

2018-01-05 Thread Ben Pfaff
This shares the infrastructure for mapping port names and numbers. It will be used in an upcoming commit. Signed-off-by: Ben Pfaff --- include/openvswitch/ofp-util.h | 33 +++-- lib/ofp-util.c | 150 ++--- 2 files changed,