* [PATCH 3/4] drivers/usb: Drop unnecessary continue
@ 2007-11-14 8:15 Julia Lawall
2007-11-27 6:48 ` Greg KH
0 siblings, 1 reply; 2+ messages in thread
From: Julia Lawall @ 2007-11-14 8:15 UTC (permalink / raw)
To: gregkh, linux-kernel, kernel-janitors
From: Julia Lawall <julia@diku.dk>
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@diku.dk>
---
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)
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [PATCH 3/4] drivers/usb: Drop unnecessary continue
2007-11-14 8:15 [PATCH 3/4] drivers/usb: Drop unnecessary continue Julia Lawall
@ 2007-11-27 6:48 ` Greg KH
0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2007-11-27 6:48 UTC (permalink / raw)
To: Julia Lawall; +Cc: gregkh, linux-kernel, kernel-janitors
On Wed, Nov 14, 2007 at 09:15:16AM +0100, Julia Lawall wrote:
> 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);
Your email client is eating all the tabs and converting them to spaces,
making these patches very hard to apply.
I've fixed this one up by hand, but please, fix your email client for
future ones.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2007-11-27 16:10 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-11-14 8:15 [PATCH 3/4] drivers/usb: Drop unnecessary continue Julia Lawall
2007-11-27 6:48 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox