[ovs-dev] [PATCH] ofproto-dpif-xlate: Remove obsolete comment.

Ben Pfaff blp at nicira.com
Fri Nov 1 21:31:11 UTC 2013


Thanks.

On Fri, Nov 01, 2013 at 02:17:11PM -0700, Alex Wang wrote:
> LGTM,
> 
> 
> On Fri, Nov 1, 2013 at 2:08 PM, Ben Pfaff <blp at nicira.com> wrote:
> 
> > Nothing about the existing code in this function cares whether Goto-Table
> > is the last action.
> >
> > Signed-off-by: Ben Pfaff <blp at nicira.com>
> > ---
> >  ofproto/ofproto-dpif-xlate.c |    1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c
> > index f19dbf1..f5bc12c 100644
> > --- a/ofproto/ofproto-dpif-xlate.c
> > +++ b/ofproto/ofproto-dpif-xlate.c
> > @@ -2529,7 +2529,6 @@ do_xlate_actions(const struct ofpact *ofpacts,
> > size_t ofpacts_len,
> >              break;
> >
> >          case OFPACT_GOTO_TABLE: {
> > -            /* It is assumed that goto-table is the last action. */
> >              struct ofpact_goto_table *ogt = ofpact_get_GOTO_TABLE(a);
> >
> >              ovs_assert(ctx->table_id < ogt->table_id);
> > --
> > 1.7.10.4
> >
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
> >



More information about the dev mailing list