On Fri, Jan 12, 2018 at 06:24:28PM -0800, Justin Pettit wrote: > > > > On Dec 31, 2017, at 9:16 PM, Ben Pfaff <b...@ovn.org> 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.
Thanks, I fixed the comment. > Acked-by: Justin Pettit <jpet...@ovn.org> Thanks! _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev