On Thu, Mar 19, 2015 at 12:08:19AM -0700, Andy Zhou wrote:
> Change ovsdb_jsonrpc_parse_monitor_request() to make
> ovsdb_monitor_table an opaque object.
>
> Signed-off-by: Andy Zhou <[email protected]>
I'm not sure that this is the best way to check for duplicates if
ovsdb_monitor is supposed to be becoming opaque, but I'm happy to deal
with that later if necessary.
It looks like the "return" statement is indented too far here in
ovsdb_monitor_table_check_duplicates():
> + for (i = 1; i < mt->n_columns; i++) {
> + if (mt->columns[i].column == mt->columns[i - 1].column) {
> + return mt->columns[i].column->name;
> + }
Acked-by: Ben Pfaff <[email protected]>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev