2014-07-14 16:30:31

by Fabian Frédérick

Subject: [PATCH 00/16] net: remove unnecessary break after goto/return

Small patchset addressing break redundancy on net branch
(suggested by Joe Perches).

Fabian Frederick (16):
packet: remove unnecessary break after return
tipc: remove unnecessary break after return
9P: remove unnecessary break after return
af_iucv: remove unnecessary break after goto
netlabel: remove unnecessary break after goto
pktgen: remove unnecessary break after goto
drop_monitor: remove unnecessary break after return
mac80211: remove unnecessary break after return
af_key: remove unnecessary break after return
netfilter: remove unnecessary break after return
ipv6: remove unnecessary break after return
NFC: remove unnecessary break after goto
caif: remove unnecessary break after goto
irda: remove unnecessary break after return
ieee802154: remove unnecessary break after goto
net: sctp: remove unnecessary break after return/goto

net/9p/client.c | 1 -
net/caif/cfctrl.c | 1 -
net/core/drop_monitor.c | 2 --
net/core/pktgen.c | 1 -
net/ieee802154/6lowpan_iphc.c | 1 -
net/ipv6/ipv6_sockglue.c | 1 -
net/ipv6/netfilter/ip6t_ipv6header.c | 1 -
net/irda/af_irda.c | 1 -
net/irda/irlan/irlan_common.c | 1 -
net/iucv/af_iucv.c | 1 -
net/key/af_key.c | 3 ---
net/mac80211/mesh_hwmp.c | 1 -
net/netlabel/netlabel_kapi.c | 1 -
net/nfc/digital_dep.c | 2 --
net/packet/af_packet.c | 2 --
net/sctp/outqueue.c | 1 -
net/sctp/sm_statefuns.c | 4 ----
net/tipc/socket.c | 1 -
18 files changed, 26 deletions(-)

--
1.8.4.5

2014-07-14 16:32:22

by Fabian Frédérick

Subject: [PATCH 08/16] mac80211: remove unnecessary break after return

Signed-off-by: Fabian Frederick <[email protected]>
---
net/mac80211/mesh_hwmp.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/net/mac80211/mesh_hwmp.c b/net/mac80211/mesh_hwmp.c
index 94758b9..214e63b 100644
--- a/net/mac80211/mesh_hwmp.c
+++ b/net/mac80211/mesh_hwmp.c
@@ -157,7 +157,6 @@ static int mesh_path_sel_frame_tx(enum mpath_frame_type action, u8 flags,
default:
kfree_skb(skb);
return -ENOTSUPP;
- break;
}
*pos++ = ie_len;
*pos++ = flags;
--
1.8.4.5

2014-07-15 23:27:34

by David Miller

Subject: Re: [PATCH 00/16] net: remove unnecessary break after goto/return

From: Fabian Frederick <[email protected]>
Date: Mon, 14 Jul 2014 18:30:50 +0200

> Small patchset addressing break redundancy on net branch
> (suggested by Joe Perches).

Series applied, thanks Fabian.