Thanks, applied to master.

On Mon, Apr 29, 2013 at 06:04:01PM -0700, Justin Pettit wrote:
> Looks good.
> 
> --Justin
> 
> 
> On Apr 29, 2013, at 5:49 PM, Ben Pfaff <b...@nicira.com> wrote:
> 
> > Um, I mean, remove useless code.
> > 
> > Signed-off-by: Ben Pfaff <b...@nicira.com>
> > ---
> > 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
> > dev@openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
> 
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to