From: Julia Lawall <julia@xxxxxxx> Continue is not needed at the bottom of a loop. The semantic patch implementing this change is as follows: @@ @@ for (...;...;...) { ... if (...) { ... - continue; } } Signed-off-by: Julia Lawall <julia@xxxxxxx> --- diff -u -p -b -B a/drivers/usb/host/isp116x-hcd.c b/drivers/usb/host/isp116x-hcd.c --- a/drivers/usb/host/isp116x-hcd.c 2007-10-22 11:25:24.000000000 +0200 +++ b/drivers/usb/host/isp116x-hcd.c 2007-11-13 23:14:10.000000000 +0100 @@ -918,7 +918,6 @@ static int isp116x_hub_status_data(struc | RH_PS_OCIC | RH_PS_PRSC)) { changed = 1; buf[0] |= 1 << (i + 1); - continue; } } spin_unlock_irqrestore(&isp116x->lock, flags); diff -u -p -b -B a/drivers/usb/image/mdc800.c b/drivers/usb/image/mdc800.c --- a/drivers/usb/image/mdc800.c 2007-07-23 11:27:17.000000000 +0200 +++ b/drivers/usb/image/mdc800.c 2007-11-13 23:14:33.000000000 +0100 @@ -479,7 +479,6 @@ static int mdc800_usb_probe (struct usb_ irq_interval=intf_desc->endpoint [j].desc.bInterval; } - continue; } } if (mdc800->endpoint[i] == -1) - To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html