On Sun, Jul 12, 2015 at 09:25:14PM -0700, Justin Pettit wrote:
> 
> > On Jul 12, 2015, at 9:04 PM, Ben Pfaff <[email protected]> wrote:
> > 
> >  * This function is appropriate when 'hmap' will soon have about as many
> >  * elements as it before.  If 'hmap' will likely have fewer elements than
> 
> I think there's a "did" missing before "before". 

Thanks, fixed also.

> > - * before, use hmap_destroy() followed by hmap_clear() to save memory and
> > + * before, use hmap_destroy() followed by hmap_init() to save memory and
> >  * iteration time. */
> 
> This seems like more than a typo, but:
> 
> Acked-by: Justin Pettit <[email protected]>

That's true, I changed the title to refer to bad advice rather than just
a typo.

Applied to master, thanks!
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to