Re: [ovs-dev] [PATCH] cmap: Merge CMAP_FOR_EACH_SAFE into CMAP_FOR_EACH.

2014-07-29 Thread Ben Pfaff
On Mon, Jul 28, 2014 at 02:03:49PM -0700, Jarno Rajahalme wrote: > Ben, > > Thanks for reminding, somehow I lost track of this. > > This is actually a nice cleanup :-) > > Acked-by: Jarno Rajahalme Thanks, applied to master. ___ dev mailing list dev@

Re: [ovs-dev] [PATCH] cmap: Merge CMAP_FOR_EACH_SAFE into CMAP_FOR_EACH.

2014-07-28 Thread Jarno Rajahalme
Ben, Thanks for reminding, somehow I lost track of this. This is actually a nice cleanup :-) Acked-by: Jarno Rajahalme On Jul 21, 2014, at 9:38 PM, Ben Pfaff wrote: > There isn't any significant downside to making cmap iteration "safe" all > the time, so this drops the _SAFE variant. > > S

Re: [ovs-dev] [PATCH] cmap: Merge CMAP_FOR_EACH_SAFE into CMAP_FOR_EACH.

2014-07-28 Thread Ben Pfaff
On Mon, Jul 21, 2014 at 09:38:27PM -0700, Ben Pfaff wrote: > There isn't any significant downside to making cmap iteration "safe" all > the time, so this drops the _SAFE variant. > > Similar changes to CMAP_CURSOR_FOR_EACH and CMAP_CURSOR_FOR_EACH_CONTINUE. > > Signed-off-by: Ben Pfaff Jarno, y

[ovs-dev] [PATCH] cmap: Merge CMAP_FOR_EACH_SAFE into CMAP_FOR_EACH.

2014-07-21 Thread Ben Pfaff
There isn't any significant downside to making cmap iteration "safe" all the time, so this drops the _SAFE variant. Similar changes to CMAP_CURSOR_FOR_EACH and CMAP_CURSOR_FOR_EACH_CONTINUE. Signed-off-by: Ben Pfaff --- lib/classifier.c | 4 ++-- lib/cmap.h| 57 ---