On Thu, Aug 10, 2017 at 01:01:32PM +0800, Gao Zhenyu wrote:
> How about:
> struct ovsdb_idl_table {
> ...
>     const struct ovsdb_idl_table_class *table_class
> ....
> }
> 
> struct ovsdb_idl {
> ....
>      const struct ovsdb_idl_class *idl_class;
> ....

Why make it longer?

> Besides of that, I see many places consume the table class.
> Do you mind to make a macro helps to fetch the class?
> Like:
> 
> #define OVSDB_GET_TABLE_CLASS(row) \
>             ((row)->table->class)

Ugh.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to