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