> On Aug 7, 2016, at 5:44 AM, Ryan Moats <rmo...@us.ibm.com> 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>
>
Thanks for your review. This patch makes only the codes clearer, and doesn’t
make
much sense.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
