On Mon, Feb 22, 2016 at 07:58:22PM +0000, Lutz, Arnoldo wrote:
> This commit adds some definitions and skeleton functions to start working.
> Ovsdb tests run and pass ok.
> 
> Signed-off-by: arnoldo.lutz.guev...@hpe.com <arnoldo.lutz.guev...@hpe.com>

This adds some compiler warnings:

../lib/ovsdb-idl.c:3221:62: error: unused parameter 'row'
      [-Werror,-Wunused-parameter]
ovsdb_idl_txn_delete_partial_map(const struct ovsdb_idl_row *row,
                                                             ^
../lib/ovsdb-idl.c:3222:64: error: unused parameter 'column'
      [-Werror,-Wunused-parameter]
                                const struct ovsdb_idl_column *column,
                                                               ^
../lib/ovsdb-idl.c:3223:53: error: unused parameter 'datum'
      [-Werror,-Wunused-parameter]
                                struct ovsdb_datum *datum)

This adds some new jargon about "partial maps" but it doesn't define it
anywhere.  There should be a definition so that readers of the code
don't have to guess what it's about.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to