On Sat, Aug 06, 2016 at 04:44:13PM -0500, Ryan Moats wrote:
> "dev" <[email protected]> wrote on 08/04/2016 06:50:03 AM:
> 
> > From: nickcooper-zhangtonghao <[email protected]>
> > To: [email protected]
> > Cc: nickcooper-zhangtonghao <[email protected]>
> > Date: 08/04/2016 06:50 AM
> > Subject: [ovs-dev] [PATCH] ovn-controller:improve the ofctrl_run,
> pinctrl_run
> > Sent by: "dev" <[email protected]>
> >
> > 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-
> > [email protected]>
> > ---
> 
> 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 <[email protected]>

I used my discretion to not add that to the commit message before
applying ;-)
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to