Looks good to me, thanks. Reviewed-by: Yifeng Sun <pkusunyif...@gmail.com>
On Mon, Sep 24, 2018 at 9:33 PM Ben Pfaff <b...@ovn.org> wrote: > The call to ovsdb_datum_diff() initializes 'new', so it's not necessary to > also do it in ovsdb_datum_apply_diff(). > > Found by inspection. > > Signed-off-by: Ben Pfaff <b...@ovn.org> > --- > lib/ovsdb-data.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/lib/ovsdb-data.c b/lib/ovsdb-data.c > index 1e523f29cb87..08b94cafa737 100644 > --- a/lib/ovsdb-data.c > +++ b/lib/ovsdb-data.c > @@ -2121,7 +2121,6 @@ ovsdb_datum_apply_diff(struct ovsdb_datum *new, > const struct ovsdb_datum *diff, > const struct ovsdb_type *type) > { > - ovsdb_datum_init_empty(new); > ovsdb_datum_diff(new, old, diff, type); > > /* Make sure member size of 'new' conforms to type. */ > -- > 2.16.1 > > _______________________________________________ > dev mailing list > d...@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev