Looks good. --Justin
On Apr 29, 2013, at 5:49 PM, Ben Pfaff <[email protected]> wrote: > Um, I mean, remove useless code. > > Signed-off-by: Ben Pfaff <[email protected]> > --- > lib/dpif-netdev.c | 4 ---- > 1 files changed, 0 insertions(+), 4 deletions(-) > > diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c > index 6838800..632a1de 100644 > --- a/lib/dpif-netdev.c > +++ b/lib/dpif-netdev.c > @@ -493,7 +493,6 @@ static int > do_del_port(struct dp_netdev *dp, uint32_t port_no) > { > struct dp_netdev_port *port; > - char *name; > int error; > > error = get_port_by_number(dp, port_no, &port); > @@ -505,11 +504,8 @@ do_del_port(struct dp_netdev *dp, uint32_t port_no) > dp->ports[port->port_no] = NULL; > dp->serial++; > > - name = xstrdup(netdev_vport_get_dpif_port(port->netdev)); > netdev_close(port->netdev); > free(port->type); > - > - free(name); > free(port); > > return 0; > -- > 1.7.2.5 > > _______________________________________________ > dev mailing list > [email protected] > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
