> On Dec 31, 2017, at 9:16 PM, Ben Pfaff <[email protected]> wrote: > > diff --git a/ovsdb/monitor.c b/ovsdb/monitor.c > index b2ecd109ed60..3e58c3fbd274 100644 > --- a/ovsdb/monitor.c > +++ b/ovsdb/monitor.c > > +void > +ovsdb_monitors_remove(struct ovsdb *db) > +{ > + struct ovsdb_monitor *m, *next_m; > + > + LIST_FOR_EACH_SAFE (m, next_m, list_node, &db->monitors) { > + struct jsonrpc_monitor_node *jm, *next_jm; > + > + /* Delete all front end monitors. Removing the last front > + * end monitor will also destroy the corresponding 'ovsdb_monitor'. > + * ovsdb monitor will also be destroied. */
I found the last sentence confusing based on the previous sentence. Also, "destroyed" is misspelled. Acked-by: Justin Pettit <[email protected]> --Justin _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
