On Sat, Aug 06, 2016 at 04:44:13PM -0500, Ryan Moats wrote: > "dev" <dev-boun...@openvswitch.org> wrote on 08/04/2016 06:50:03 AM: > > > From: nickcooper-zhangtonghao <nickcooper-zhangtong...@opencloud.tech> > > To: dev@openvswitch.org > > Cc: nickcooper-zhangtonghao <nickcooper-zhangtong...@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-boun...@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- > > zhangtong...@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 <rmo...@us.ibm.com>
I used my discretion to not add that to the commit message before applying ;-) _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev