> On Oct 9, 2015, at 9:15 PM, Ben Pfaff <[email protected]> wrote: > > + /* Add any patch ports that should exist but don't. */ > + parse_bridge_mappings(ctx, br_int, &existing_ports);
This comment could probably use a bit more information. > + /* Delete any patch ports that do exist but shouldn't. (Any that both > + * should and do exist were removed above.) */ I think it might be a bit clearer if you replaced "above" with 'from "existing_ports"' or something similar. Acked-by: Justin Pettit <[email protected]> --Justin _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
