> +                            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

Reply via email to