> + tag_set_add(&o->revalidate_set, e->tag); > + mac_learning_expire (o->ml, e);
Redundant space in this function call. As I read this, I was wondering if we could somehow expose the all_ofprotos map in ofproto.c to providers by implementing some sort of iterator at that level. The current code is pretty simple and straightforward, so it may not be worth it. Looks good either way. Ethan _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev