[PATCH] net/udp_offload: Drop unnecessary continue

From: Himangi Saraogi
Date: Wed Aug 13 2014 - 05:16:16 EST


Continue is not needed at the bottom of a loop.

The Coccinelle semantic patch implementing this change is:

@@
@@

for (...;...;...) {
...
if (...) {
...
- continue;
}
}

Signed-off-by: Himangi Saraogi <himangi774@xxxxxxxxx>
Acked-by: Julia Lawall <julia.lawall@xxxxxxx>
---
net/ipv4/udp_offload.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/net/ipv4/udp_offload.c b/net/ipv4/udp_offload.c
index 59035bc..62d5b9b 100644
--- a/net/ipv4/udp_offload.c
+++ b/net/ipv4/udp_offload.c
@@ -269,10 +269,8 @@ unflush:
continue;

uh2 = (struct udphdr *)(p->data + off);
- if ((*(u32 *)&uh->source != *(u32 *)&uh2->source)) {
+ if ((*(u32 *)&uh->source != *(u32 *)&uh2->source))
NAPI_GRO_CB(p)->same_flow = 0;
- continue;
- }
}

skb_gro_pull(skb, sizeof(struct udphdr)); /* pull encapsulating udp header */
--
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/