Looks good. Ethan
On Fri, Jan 13, 2012 at 16:43, Ben Pfaff <b...@nicira.com> wrote: > Nothing uses op->victim following this assignment, so it is not > needed. > > Signed-off-by: Ben Pfaff <b...@nicira.com> > --- > ofproto/ofproto.c | 1 - > 1 files changed, 0 insertions(+), 1 deletions(-) > > diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c > index ff356f4..cc309b3 100644 > --- a/ofproto/ofproto.c > +++ b/ofproto/ofproto.c > @@ -3297,7 +3297,6 @@ ofoperation_complete(struct ofoperation *op, enum > ofperr error) > oftable_substitute_rule(rule, op->victim); > ofproto_rule_destroy__(rule); > } > - op->victim = NULL; > break; > > case OFOPERATION_DELETE: > -- > 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
