[ovs-dev] [flow monitor 01/11] ofproto: Fix typos in comments.

Ben Pfaff blp at nicira.com
Thu Jul 5 23:00:11 UTC 2012


Thanks.

By the way, I'm going to send out a new version of patches 10 and 11,
possibly today, so don't bother reviewing those yet.  Patch 8 already
got committed earlier today (there was another series that depended on
it), so you can skip that one too.  The rest of the series won't
really change.

On Thu, Jul 05, 2012 at 03:56:45PM -0700, Justin Pettit wrote:
> Looks good.
> 
> --Justin
> 
> 
> On Jun 29, 2012, at 10:40 PM, Ben Pfaff wrote:
> 
> > This corrects the spelling of the OFOPERATION_* constants.
> > 
> > Signed-off-by: Ben Pfaff <blp at nicira.com>
> > ---
> > ofproto/ofproto.c |    6 +++---
> > 1 files changed, 3 insertions(+), 3 deletions(-)
> > 
> > diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
> > index ce4da9d..5221318 100644
> > --- a/ofproto/ofproto.c
> > +++ b/ofproto/ofproto.c
> > @@ -117,9 +117,9 @@ struct ofoperation {
> >     struct hmap_node hmap_node; /* In ofproto's "deletions" hmap. */
> >     struct rule *rule;          /* Rule being operated upon. */
> >     enum ofoperation_type type; /* Type of operation. */
> > -    struct rule *victim;        /* OFOPERATION_ADDING: Replaced rule. */
> > -    union ofp_action *actions;  /* OFOPERATION_MODIFYING: Replaced actions. */
> > -    int n_actions;              /* OFOPERATION_MODIFYING: # of old actions. */
> > +    struct rule *victim;        /* OFOPERATION_ADD: Replaced rule. */
> > +    union ofp_action *actions;  /* OFOPERATION_MODIFY: Replaced actions. */
> > +    int n_actions;              /* OFOPERATION_MODIFY: # of old actions. */
> >     ovs_be64 flow_cookie;       /* Rule's old flow cookie. */
> > };
> > 
> > -- 
> > 1.7.2.5
> > 
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
> 



More information about the dev mailing list