From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:45603) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QInqG-0000W5-8q for qemu-devel@nongnu.org; Sat, 07 May 2011 16:11:08 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QInqF-0002Bm-8c for qemu-devel@nongnu.org; Sat, 07 May 2011 16:11:08 -0400 Received: from moutng.kundenserver.de ([212.227.126.187]:63211) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QInqE-0002AW-UC for qemu-devel@nongnu.org; Sat, 07 May 2011 16:11:07 -0400 From: Stefan Weil Date: Sat, 7 May 2011 22:10:53 +0200 Message-Id: <1304799053-19137-1-git-send-email-weil@mail.berlios.de> Subject: [Qemu-devel] [PATCH] usb-linux: Add missing break statement List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: QEMU Developers Cc: Gerd Hoffmann , Hans de Goede cppcheck report: usb-linux.c:661: warning: Redundant assignment of "len" in switch Please check whether adding a break statement is the correct solution for this warning. Cc: Hans de Goede Cc: Gerd Hoffmann Signed-off-by: Stefan Weil --- usb-linux.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/usb-linux.c b/usb-linux.c index 36a01ea..0ef1d26 100644 --- a/usb-linux.c +++ b/usb-linux.c @@ -659,6 +659,7 @@ static int usb_host_handle_iso_data(USBHostDevice *s, USBPacket *p, int in) switch(errno) { case ETIMEDOUT: len = USB_RET_NAK; + break; case EPIPE: default: len = USB_RET_STALL; -- 1.7.2.5