Patchwork [5/8] net/atm/lec.c: drop code after return

login
register
mail settings
Submitter Andrew Morton
Date Sept. 22, 2008, 9:50 p.m.
Message ID <200809222150.m8MLofR7031869@imap1.linux-foundation.org>
Download mbox | patch
Permalink /patch/973/
State Accepted
Delegated to: David Miller
Headers show

Comments

Andrew Morton - Sept. 22, 2008, 9:50 p.m.
From: Julia Lawall <julia@diku.dk>

The break after the return serves no purpose.

Signed-off-by: Julia Lawall <julia@diku.dk>
Reviewed-by: Richard Genoud <richard.genoud@gmail.com>
Cc: Chas Williams <chas@cmf.nrl.navy.mil>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---

 net/atm/lec.c |    1 -
 1 file changed, 1 deletion(-)
David Miller - Sept. 23, 2008, 2:24 a.m.
From: akpm@linux-foundation.org
Date: Mon, 22 Sep 2008 14:50:41 -0700

> From: Julia Lawall <julia@diku.dk>
> 
> The break after the return serves no purpose.
> 
> Signed-off-by: Julia Lawall <julia@diku.dk>
> Reviewed-by: Richard Genoud <richard.genoud@gmail.com>
> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>

Applied to net-next-2.6
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff -puN net/atm/lec.c~net-atm-lecc-drop-code-after-return net/atm/lec.c
--- a/net/atm/lec.c~net-atm-lecc-drop-code-after-return
+++ a/net/atm/lec.c
@@ -1931,7 +1931,6 @@  static struct atm_vcc *lec_arp_resolve(s
 		switch (priv->lane_version) {
 		case 1:
 			return priv->mcast_vcc;
-			break;
 		case 2:	/* LANE2 wants arp for multicast addresses */
 			if (!compare_ether_addr(mac_to_find, bus_mac))
 				return priv->mcast_vcc;