Re: [ovs-dev] [PATCH] classifier: Use HMAP_FOR_EACH, HMAP_FOR_EACH_CONTINUE.

2011-11-11 Thread Ben Pfaff
Thanks, I pushed this. On Thu, Nov 10, 2011 at 11:12:01AM -0800, Ethan Jackson wrote: > Looks good. > > Ethan > > On Wed, Nov 9, 2011 at 16:24, Ben Pfaff wrote: > > I like how this removes over 20 lines of code and ends up more readable. > > --- > > ?lib/classifier.c | ? 30 +++-

Re: [ovs-dev] [PATCH] classifier: Use HMAP_FOR_EACH, HMAP_FOR_EACH_CONTINUE.

2011-11-10 Thread Ethan Jackson
Looks good. Ethan On Wed, Nov 9, 2011 at 16:24, Ben Pfaff wrote: > I like how this removes over 20 lines of code and ends up more readable. > --- >  lib/classifier.c |   30 +++--- >  1 files changed, 3 insertions(+), 27 deletions(-) > > diff --git a/lib/classifier.c b/lib

[ovs-dev] [PATCH] classifier: Use HMAP_FOR_EACH, HMAP_FOR_EACH_CONTINUE.

2011-11-09 Thread Ben Pfaff
I like how this removes over 20 lines of code and ends up more readable. --- lib/classifier.c | 30 +++--- 1 files changed, 3 insertions(+), 27 deletions(-) diff --git a/lib/classifier.c b/lib/classifier.c index 8ffc96f..04e545e 100644 --- a/lib/classifier.c +++ b/lib/cl