Ben Pfaff <[email protected]> wrote on 06/23/2016 10:58:03 PM: > From: Ben Pfaff <[email protected]> > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: [email protected] > Date: 06/23/2016 10:58 PM > Subject: Re: [ovs-dev] [PATCH] ovn-controller: Use new ovsdb-idl > helpers to make logic more readable. > > On Thu, Jun 23, 2016 at 10:48:31PM -0500, Ryan Moats wrote: > > Ben Pfaff <[email protected]> wrote on 06/23/2016 10:33:53 PM: > > > > > From: Ben Pfaff <[email protected]> > > > To: Ryan Moats/Omaha/IBM@IBMUS > > > Cc: [email protected] > > > Date: 06/23/2016 10:34 PM > > > Subject: Re: [ovs-dev] [PATCH] ovn-controller: Use new ovsdb-idl > > > helpers to make logic more readable. > > > > > > On Thu, Jun 23, 2016 at 08:01:52PM -0500, Ryan Moats wrote: > > > > "dev" <[email protected]> wrote on 06/23/2016 07:00:51 PM: > > > > > > > > > From: Ben Pfaff <[email protected]> > > > > > To: [email protected] > > > > > Cc: Ben Pfaff <[email protected]> > > > > > Date: 06/23/2016 07:01 PM > > > > > Subject: [ovs-dev] [PATCH] ovn-controller: Use new ovsdb-idl helpers > > > > > to make logic more readable. > > > > > Sent by: "dev" <[email protected]> > > > > > > > > > > Also there were lots of 'continue's sprinkled around that didn't seem > > to > > > > > be needed given some simple code rearrangement. > > > > > > > > > > Signed-off-by: Ben Pfaff <[email protected]> > > > > > > > > I'll not only ack this, I'll pull it in and base my residual > > incremental > > > > processing patches on it because it makes it *MUCH* cleaner: > > > > > > > > Acked-by: Ryan Moats <[email protected]> > > > > > > > > In fact .... Happily-acked-by :) > > > > > > Such enthusiasm! And speed! > > > > > > Thanks, I applied this to master. > > > > > > > Are you kidding, I was thinking "man, I should have thought of that!!!" > > I swear I suggested it once. At this point I figured an example would > make it more obvious ;-) >
You probably did and I misunderstood you... Incoming v19 before I call it a night - I have all the changes/rebases ready to roll... Ryan _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
