From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Ns0RD-00088R-Gv for qemu-devel@nongnu.org; Wed, 17 Mar 2010 17:05:59 -0400 Received: from [199.232.76.173] (port=37953 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Ns0RC-00087j-Up for qemu-devel@nongnu.org; Wed, 17 Mar 2010 17:05:58 -0400 Received: from Debian-exim by monty-python.gnu.org with spam-scanned (Exim 4.60) (envelope-from ) id 1Ns0RC-0008Ro-4p for qemu-devel@nongnu.org; Wed, 17 Mar 2010 17:05:58 -0400 Received: from mail-px0-f200.google.com ([209.85.216.200]:34410) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1Ns0RB-0008Qy-Pq for qemu-devel@nongnu.org; Wed, 17 Mar 2010 17:05:57 -0400 Received: by pxi38 with SMTP id 38so727407pxi.27 for ; Wed, 17 Mar 2010 14:05:51 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <4BA142C4.5030805@codemonkey.ws> References: <1268053115.2130.4.camel@localhost.localdomain> <201003171708.04633.paul@codesourcery.com> <4BA10E49.90703@codemonkey.ws> <201003171743.06728.paul@codesourcery.com> <4BA13E6E.4060708@codemonkey.ws> <1268859417.23390.3.camel@localhost.localdomain> <4BA142C4.5030805@codemonkey.ws> Date: Wed, 17 Mar 2010 23:05:51 +0200 Message-ID: Subject: Re: [Qemu-devel] [PATCH] [TRIVIAL] usb-linux: remove unreachable default in switch statement From: Blue Swirl Content-Type: text/plain; charset=UTF-8 List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: Paul Bolle , Paul Brook , qemu-devel@nongnu.org On 3/17/10, Anthony Liguori wrote: > On 03/17/2010 03:56 PM, Paul Bolle wrote: > > > On Wed, 2010-03-17 at 15:41 -0500, Anthony Liguori wrote: > > > > > > > On 03/17/2010 03:15 PM, Blue Swirl wrote: > > > > > > > > > > This breaks build (gcc 4.3.2): > > > > CC usb-linux.o > > > > cc1: warnings being treated as errors > > > > /src/qemu/usb-linux.c: In function 'usb_linux_update_endp_table': > > > > /src/qemu/usb-linux.c:759: error: 'type' may be used uninitialized in > > > > this function > > > > > > > > > > > > > > > That's unfortunate. I'll revert. > > > > > > > > I can't reproduce this with gcc-4.4.3-8.fc13.i686 (which I'm currently > > running). The patch was tested and submitted when I was running > > gcc-4.4.3-6.fc13.i686. > > > > > > Yeah, it worked for me, but clearly older versions of gcc are less smart > about calculating ranges. Actually OpenBSD gcc (3.4.4) has no problems.