Re: [ovs-dev] [PATCH] tunnel: Make tnl_find() easier to understand.

2013-09-25 Thread Ben Pfaff
On Tue, Aug 27, 2013 at 04:03:18PM -0700, Jarno Rajahalme wrote: > > On Aug 27, 2013, at 11:12 AM, Ben Pfaff wrote: > > > Suggested-by: pritesh > > Signed-off-by: Ben Pfaff > > --- > > ofproto/tunnel.c | 100 > > +- > > 1 file changed, 46 in

Re: [ovs-dev] [PATCH] tunnel: Make tnl_find() easier to understand.

2013-08-28 Thread Jarno Rajahalme
On Aug 27, 2013, at 10:43 PM, Ben Pfaff wrote: > On Tue, Aug 27, 2013 at 04:03:18PM -0700, Jarno Rajahalme wrote: >> >> On Aug 27, 2013, at 11:12 AM, Ben Pfaff wrote: >> >>> Suggested-by: pritesh >>> Signed-off-by: Ben Pfaff >>> --- >>> ofproto/tunnel.c | 100 >>> +

Re: [ovs-dev] [PATCH] tunnel: Make tnl_find() easier to understand.

2013-08-27 Thread Ben Pfaff
On Tue, Aug 27, 2013 at 04:03:18PM -0700, Jarno Rajahalme wrote: > > On Aug 27, 2013, at 11:12 AM, Ben Pfaff wrote: > > > Suggested-by: pritesh > > Signed-off-by: Ben Pfaff > > --- > > ofproto/tunnel.c | 100 > > +- > > 1 file changed, 46 in

Re: [ovs-dev] [PATCH] tunnel: Make tnl_find() easier to understand.

2013-08-27 Thread Jarno Rajahalme
On Aug 27, 2013, at 11:12 AM, Ben Pfaff wrote: > Suggested-by: pritesh > Signed-off-by: Ben Pfaff > --- > ofproto/tunnel.c | 100 +- > 1 file changed, 46 insertions(+), 54 deletions(-) > > diff --git a/ofproto/tunnel.c b/ofproto/tunnel.c > i

Re: [ovs-dev] [PATCH] tunnel: Make tnl_find() easier to understand.

2013-08-27 Thread Ben Pfaff
Thanks. I am going to give others a day or so to look at this, since it is only a cleanup. I'll apply it then assuming I get no negative feedback. On Tue, Aug 27, 2013 at 07:01:11PM +, Pritesh Kothari (pritkoth) wrote: > Looks good and simple to understand. (passed tests also). > > Acked-by

Re: [ovs-dev] [PATCH] tunnel: Make tnl_find() easier to understand.

2013-08-27 Thread Pritesh Kothari (pritkoth)
Looks good and simple to understand. (passed tests also). Acked-by: pritesh On Aug 27, 2013, at 11:12 AM, Ben Pfaff wrote: > Suggested-by: pritesh > Signed-off-by: Ben Pfaff > --- > ofproto/tunnel.c | 100 +- > 1 file changed, 46 insertions(

[ovs-dev] [PATCH] tunnel: Make tnl_find() easier to understand.

2013-08-27 Thread Ben Pfaff
Suggested-by: pritesh Signed-off-by: Ben Pfaff --- ofproto/tunnel.c | 100 +- 1 file changed, 46 insertions(+), 54 deletions(-) diff --git a/ofproto/tunnel.c b/ofproto/tunnel.c index 202358b..4729f5f 100644 --- a/ofproto/tunnel.c +++ b/ofprot