On Fri, Sep 07, 2018 at 09:40:30AM -0700, Gregory Rose wrote: > On 9/7/2018 3:21 AM, wangyunjian wrote: > >From: Yunjian Wang <[email protected]> > > > >Signed-off-by: Yunjian Wang <[email protected]> > >--- > > datapath/linux/compat/stt.c | 3 --- > > 1 file changed, 3 deletions(-) > > > >diff --git a/datapath/linux/compat/stt.c b/datapath/linux/compat/stt.c > >index 8d16c1f..506f1d9 100644 > >--- a/datapath/linux/compat/stt.c > >+++ b/datapath/linux/compat/stt.c > >@@ -1706,9 +1706,6 @@ static void stt_cleanup(struct net *net) > > int i; > > sn->n_tunnels--; > >- if (sn->n_tunnels) > >- goto out; > >-out: > > n_tunnels--; > > if (n_tunnels) > > return; > > It's always nice to snip out some dead code. LGTM. > > Reviewed-by: Greg Rose <[email protected]>
Applied to master, thanks wangyunjian and Greg! _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
