[PATCH 5/6] linux: remove bogus setting of status from handle_iso_completion

Hans de Goede hdegoede at redhat.com
Wed Mar 21 11:09:04 EDT 2012


Remove a useless check then set of status because:
1. The check is always true; and
2. The new value status gets set to never gets used

Signed-off-by: Hans de Goede <hdegoede at redhat.com>
---
 libusb/os/linux_usbfs.c |    3 ---
 1 file changed, 3 deletions(-)

diff --git a/libusb/os/linux_usbfs.c b/libusb/os/linux_usbfs.c
index d41aac8..8d88c87 100644
--- a/libusb/os/linux_usbfs.c
+++ b/libusb/os/linux_usbfs.c
@@ -2122,9 +2122,6 @@ static int handle_iso_completion(struct usbi_transfer *itransfer,
 	if (tpriv->reap_action != NORMAL) { /* cancelled or submit_fail */
 		usbi_dbg("CANCEL: urb status %d", urb->status);
 
-		if (status == LIBUSB_TRANSFER_COMPLETED)
-			status = LIBUSB_TRANSFER_ERROR;
-
 		if (tpriv->num_retired == num_urbs) {
 			usbi_dbg("CANCEL: last URB handled, reporting");
 			free_iso_urbs(tpriv);
-- 
1.7.9.3




More information about the libusbx mailing list