diff options
| author | stephen hemminger <stephen@networkplumber.org> | 2014-02-13 08:51:22 +0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2014-02-14 03:33:38 +0400 |
| commit | 2045ceaed4d54e6e698874d008be727ee5b2a01c (patch) | |
| tree | a481bc211dd20db90275aceab3721c7a8fb147d9 /net/ipv4 | |
| parent | 45f7435968363816f8fc4c6abef692808534140d (diff) | |
| download | linux-2045ceaed4d54e6e698874d008be727ee5b2a01c.tar.xz | |
net: remove unnecessary return's
One of my pet coding style peeves is the practice of
adding extra return; at the end of function.
Kill several instances of this in network code.
I suppose some coccinelle wizardy could do this automatically.
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4')
| -rw-r--r-- | net/ipv4/route.c | 1 | ||||
| -rw-r--r-- | net/ipv4/tcp_output.c | 1 |
2 files changed, 0 insertions, 2 deletions
diff --git a/net/ipv4/route.c b/net/ipv4/route.c index 25071b48921c..271554c61276 100644 --- a/net/ipv4/route.c +++ b/net/ipv4/route.c @@ -697,7 +697,6 @@ static void update_or_create_fnhe(struct fib_nh *nh, __be32 daddr, __be32 gw, out_unlock: spin_unlock_bh(&fnhe_lock); - return; } static void __ip_do_redirect(struct rtable *rt, struct sk_buff *skb, struct flowi4 *fl4, diff --git a/net/ipv4/tcp_output.c b/net/ipv4/tcp_output.c index 3be16727f058..48414fcca973 100644 --- a/net/ipv4/tcp_output.c +++ b/net/ipv4/tcp_output.c @@ -2071,7 +2071,6 @@ rearm_timer: if (likely(!err)) NET_INC_STATS_BH(sock_net(sk), LINUX_MIB_TCPLOSSPROBES); - return; } /* Push out any pending frames which were held back due to |