On Tue, Jan 19, 2016 at 12:40:05PM -0800, Jarno Rajahalme wrote:
> With a minor function naming question below:
> 
> Acked-by: Jarno Rajahalme <ja...@ovn.org>

...

> > -put_queue_rate(struct ofpbuf *reply, enum ofp_queue_properties property,
> > -               uint16_t rate)
> > +put_ofp11_queue_rate(struct ofpbuf *reply,
> > +                     enum ofp10_queue_properties property, uint16_t rate)
> 
> It is unclear to me why this function would not be called
> ‘put_ofp10_queue_rate’ instead?

You're right, that's more accurate.  I made the change here and below.

I'll push this and the previous patch to master in a minute.

Thanks,

Ben.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to