diff options
| author | Joe Perches <joe@perches.com> | 2010-05-18 02:08:21 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-05-18 02:23:14 -0400 |
| commit | 3fa21e07e6acefa31f974d57fba2b6920a7ebd1a (patch) | |
| tree | 4254644ef2579e9ac96db6ec0535b5e4231fd18e /net | |
| parent | b60b6592baa69c43a5a0f55d6300a7feaab15338 (diff) | |
net: Remove unnecessary returns from void function()s
This patch removes from net/ (but not any netfilter files)
all the unnecessary return; statements that precede the
last closing brace of void functions.
It does not remove the returns that are immediately
preceded by a label as gcc doesn't like that.
Done via:
$ grep -rP --include=*.[ch] -l "return;\n}" net/ | \
xargs perl -i -e 'local $/ ; while (<>) { s/\n[ \t\n]+return;\n}/\n}/g; print; }'
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
48 files changed, 0 insertions, 131 deletions
diff --git a/net/9p/trans_rdma.c b/net/9p/trans_rdma.c index 041101ab4aa5..0ea20c30466c 100644 --- a/net/9p/trans_rdma.c +++ b/net/9p/trans_rdma.c | |||
| @@ -308,7 +308,6 @@ handle_recv(struct p9_client *client, struct p9_trans_rdma *rdma, | |||
| 308 | req, err, status); | 308 | req, err, status); |
| 309 | rdma->state = P9_RDMA_FLUSHING; | 309 | rdma->state = P9_RDMA_FLUSHING; |
| 310 | client->status = Disconnected; | 310 | client->status = Disconnected; |
| 311 | return; | ||
| 312 | } | 311 | } |
| 313 | 312 | ||
| 314 | static void | 313 | static void |
diff --git a/net/atm/br2684.c b/net/atm/br2684.c index d6c7ceaf13e9..6719af6a59fa 100644 --- a/net/atm/br2684.c +++ b/net/atm/br2684.c | |||
| @@ -446,7 +446,6 @@ error: | |||
| 446 | net_dev->stats.rx_errors++; | 446 | net_dev->stats.rx_errors++; |
| 447 | free_skb: | 447 | free_skb: |
| 448 | dev_kfree_skb(skb); | 448 | dev_kfree_skb(skb); |
| 449 | return; | ||
| 450 | } | 449 | } |
| 451 | 450 | ||
| 452 | /* | 451 | /* |
diff --git a/net/atm/lec.c b/net/atm/lec.c index feeaf5718472..d98bde1a0ac8 100644 --- a/net/atm/lec.c +++ b/net/atm/lec.c | |||
| @@ -161,8 +161,6 @@ static void lec_handle_bridge(struct sk_buff *skb, struct net_device *dev) | |||
| 161 | skb_queue_tail(&sk->sk_receive_queue, skb2); | 161 | skb_queue_tail(&sk->sk_receive_queue, skb2); |
| 162 | sk->sk_data_ready(sk, skb2->len); | 162 | sk->sk_data_ready(sk, skb2->len); |
| 163 | } | 163 | } |
| 164 | |||
| 165 | return; | ||
| 166 | } | 164 | } |
| 167 | #endif /* defined(CONFIG_BRIDGE) || defined(CONFIG_BRIDGE_MODULE) */ | 165 | #endif /* defined(CONFIG_BRIDGE) || defined(CONFIG_BRIDGE_MODULE) */ |
| 168 | 166 | ||
| @@ -640,7 +638,6 @@ static void lec_set_multicast_list(struct net_device *dev) | |||
| 640 | * by default, all multicast frames arrive over the bus. | 638 | * by default, all multicast frames arrive over the bus. |
| 641 | * eventually support selective multicast service | 639 | * eventually support selective multicast service |
| 642 | */ | 640 | */ |
| 643 | return; | ||
| 644 | } | 641 | } |
| 645 | 642 | ||
| 646 | static const struct net_device_ops lec_netdev_ops = { | 643 | static const struct net_device_ops lec_netdev_ops = { |
| @@ -1199,8 +1196,6 @@ static void __exit lane_module_cleanup(void) | |||
| 1199 | dev_lec[i] = NULL; | 1196 | dev_lec[i] = NULL; |
| 1200 | } | 1197 | } |
| 1201 | } | 1198 | } |
| 1202 | |||
| 1203 | return; | ||
| 1204 | } | 1199 | } |
| 1205 | 1200 | ||
| 1206 | module_init(lane_module_init); | 1201 | module_init(lane_module_init); |
| @@ -1334,7 +1329,6 @@ static void lane2_associate_ind(struct net_device *dev, const u8 *mac_addr, | |||
| 1334 | priv->lane2_ops->associate_indicator(dev, mac_addr, | 1329 | priv->lane2_ops->associate_indicator(dev, mac_addr, |
| 1335 | tlvs, sizeoftlvs); | 1330 | tlvs, sizeoftlvs); |
| 1336 | } | 1331 | } |
| 1337 | return; | ||
| 1338 | } | 1332 | } |
| 1339 | 1333 | ||
| 1340 | /* | 1334 | /* |
diff --git a/net/atm/mpc.c b/net/atm/mpc.c index 436f2e177657..622b471e14e0 100644 --- a/net/atm/mpc.c +++ b/net/atm/mpc.c | |||
| @@ -455,7 +455,6 @@ static void lane2_assoc_ind(struct net_device *dev, const u8 *mac_addr, | |||
| 455 | if (end_of_tlvs - tlvs != 0) | 455 | if (end_of_tlvs - tlvs != 0) |
| 456 | pr_info("(%s) ignoring %Zd bytes of trailing TLV garbage\n", | 456 | pr_info("(%s) ignoring %Zd bytes of trailing TLV garbage\n", |
| 457 | dev->name, end_of_tlvs - tlvs); | 457 | dev->name, end_of_tlvs - tlvs); |
| 458 | return; | ||
| 459 | } | 458 | } |
| 460 | 459 | ||
| 461 | /* | 460 | /* |
| @@ -684,8 +683,6 @@ static void mpc_vcc_close(struct atm_vcc *vcc, struct net_device *dev) | |||
| 684 | 683 | ||
| 685 | if (in_entry == NULL && eg_entry == NULL) | 684 | if (in_entry == NULL && eg_entry == NULL) |
| 686 | dprintk("(%s) unused vcc closed\n", dev->name); | 685 | dprintk("(%s) unused vcc closed\n", dev->name); |
| 687 | |||
| 688 | return; | ||
| 689 | } | 686 | } |
| 690 | 687 | ||
| 691 | static void mpc_push(struct atm_vcc *vcc, struct sk_buff *skb) | 688 | static void mpc_push(struct atm_vcc *vcc, struct sk_buff *skb) |
| @@ -783,8 +780,6 @@ static void mpc_push(struct atm_vcc *vcc, struct sk_buff *skb) | |||
| 783 | 780 | ||
| 784 | memset(ATM_SKB(skb), 0, sizeof(struct atm_skb_data)); | 781 | memset(ATM_SKB(skb), 0, sizeof(struct atm_skb_data)); |
| 785 | netif_rx(new_skb); | 782 | netif_rx(new_skb); |
| 786 | |||
| 787 | return; | ||
| 788 | } | 783 | } |
| 789 | 784 | ||
| 790 | static struct atmdev_ops mpc_ops = { /* only send is required */ | 785 | static struct atmdev_ops mpc_ops = { /* only send is required */ |
| @@ -873,8 +868,6 @@ static void send_set_mps_ctrl_addr(const char *addr, struct mpoa_client *mpc) | |||
| 873 | mesg.type = SET_MPS_CTRL_ADDR; | 868 | mesg.type = SET_MPS_CTRL_ADDR; |
| 874 | memcpy(mesg.MPS_ctrl, addr, ATM_ESA_LEN); | 869 | memcpy(mesg.MPS_ctrl, addr, ATM_ESA_LEN); |
| 875 | msg_to_mpoad(&mesg, mpc); | 870 | msg_to_mpoad(&mesg, mpc); |
| 876 | |||
| 877 | return; | ||
| 878 | } | 871 | } |
| 879 | 872 | ||
| 880 | static void mpoad_close(struct atm_vcc *vcc) | 873 | static void mpoad_close(struct atm_vcc *vcc) |
| @@ -911,8 +904,6 @@ static void mpoad_close(struct atm_vcc *vcc) | |||
| 911 | pr_info("(%s) going down\n", | 904 | pr_info("(%s) going down\n", |
| 912 | (mpc->dev) ? mpc->dev->name : "<unknown>"); | 905 | (mpc->dev) ? mpc->dev->name : "<unknown>"); |
| 913 | module_put(THIS_MODULE); | 906 | module_put(THIS_MODULE); |
| 914 | |||
| 915 | return; | ||
| 916 | } | 907 | } |
| 917 | 908 | ||
| 918 | /* | 909 | /* |
| @@ -1122,7 +1113,6 @@ static void MPOA_trigger_rcvd(struct k_message *msg, struct mpoa_client *mpc) | |||
