[ovs-dev] [PATCH] ovn-controller:improve the ofctrl_run, pinctrl_run

[Ben Pfaff]

On Sat, Aug 06, 2016 at 04:44:13PM -0500, Ryan Moats wrote:
> "dev" <dev-bounces at openvswitch.org> wrote on 08/04/2016 06:50:03 AM:
> 
> > From: nickcooper-zhangtonghao <nickcooper-zhangtonghao at opencloud.tech>
> > To: dev at openvswitch.org
> > Cc: nickcooper-zhangtonghao <nickcooper-zhangtonghao at opencloud.tech>
> > Date: 08/04/2016 06:50 AM
> > Subject: [ovs-dev] [PATCH] ovn-controller:improve the ofctrl_run,
> pinctrl_run
> > Sent by: "dev" <dev-bounces at openvswitch.org>
> >
> > Before calling the function "ofctrl_run" and "pinctrl_run",
> > the "br-int"" has been checked. Remove the conditional statements in
> > the function may make the code clearer.
> >
> > Signed-off-by: nickcooper-zhangtonghao <nickcooper-
> > zhangtonghao at opencloud.tech>
> > ---
> 
> This applies cleanly, it passes all the unit tests, and I understand
> the logic behind it, but I can't help but think it means we've missed
> something along the way...
> 
> so, Reluctantly-Acked-by: Ryan Moats <rmoats at us.ibm.com>

I used my discretion to not add that to the commit message before
applying ;-)