Thanks, applied.

On Wed, May 28, 2014 at 11:35:12AM -0700, Jarno Rajahalme wrote:
> Acked-by: Jarno Rajahalme <[email protected]>
> 
> On May 28, 2014, at 11:11 AM, Ben Pfaff <[email protected]> wrote:
> 
> > This comment seems misleading, because it implies that the struct ends in
> > a flexible array member when it doesn't.
> > 
> > Signed-off-by: Ben Pfaff <[email protected]>
> > ---
> > ofproto/ofproto.c |    2 --
> > 1 file changed, 2 deletions(-)
> > 
> > diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
> > index 63f65c3..b6b536d 100644
> > --- a/ofproto/ofproto.c
> > +++ b/ofproto/ofproto.c
> > @@ -5032,8 +5032,6 @@ handle_flow_monitor_cancel(struct ofconn *ofconn, 
> > const struct ofp_header *oh)
> > /* Meters implementation.
> >  *
> >  * Meter table entry, indexed by the OpenFlow meter_id.
> > - * These are always dynamically allocated to allocate enough space for
> > - * the bands.
> >  * 'created' is used to compute the duration for meter stats.
> >  * 'list rules' is needed so that we can delete the dependent rules when the
> >  * meter table entry is deleted.
> > -- 
> > 1.7.10.4
> > 
> > _______________________________________________
> > dev mailing list
> > [email protected]
> > http://openvswitch.org/mailman/listinfo/dev
> 
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to