diff options
author | stephen hemminger <stephen@networkplumber.org> | 2014-02-12 23:51:22 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-02-13 18:33:38 -0500 |
commit | 2045ceaed4d54e6e698874d008be727ee5b2a01c (patch) | |
tree | a481bc211dd20db90275aceab3721c7a8fb147d9 /net/ceph | |
parent | 45f7435968363816f8fc4c6abef692808534140d (diff) |
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/ceph')
-rw-r--r-- | net/ceph/osd_client.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/net/ceph/osd_client.c b/net/ceph/osd_client.c index 0676f2b199d6..82750f915865 100644 --- a/net/ceph/osd_client.c +++ b/net/ceph/osd_client.c | |||
@@ -2082,7 +2082,6 @@ bad: | |||
2082 | pr_err("osdc handle_map corrupt msg\n"); | 2082 | pr_err("osdc handle_map corrupt msg\n"); |
2083 | ceph_msg_dump(msg); | 2083 | ceph_msg_dump(msg); |
2084 | up_write(&osdc->map_sem); | 2084 | up_write(&osdc->map_sem); |
2085 | return; | ||
2086 | } | 2085 | } |
2087 | 2086 | ||
2088 | /* | 2087 | /* |
@@ -2281,7 +2280,6 @@ done_err: | |||
2281 | 2280 | ||
2282 | bad: | 2281 | bad: |
2283 | pr_err("osdc handle_watch_notify corrupt msg\n"); | 2282 | pr_err("osdc handle_watch_notify corrupt msg\n"); |
2284 | return; | ||
2285 | } | 2283 | } |
2286 | 2284 | ||
2287 | /* | 2285 | /* |