Date: Fri, 17 Oct 2008 15:47:58 -0700 (PDT)
From: Linus Torvalds <torvalds@linux-foundation.org>
To: Greg KH <gregkh@suse.de>
cc: Andrew Morton <akpm@linux-foundation.org>, linux-kernel@vger.kernel.org,
       linux-usb@vger.kernel.org
Subject: Re: [GIT PATCH] USB patches for 2.6-git
In-Reply-To: <20081017220027.GA3659@kroah.com>
Message-ID: <alpine.LFD.2.00.0810171544320.3438@nehalem.linux-foundation.org>
References: <20081017220027.GA3659@kroah.com>
User-Agent: Alpine 2.00 (LFD 1167 2008-08-23)
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=ISO-8859-7
Content-Transfer-Encoding: 8BIT
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1154
Lines: 29


On Fri, 17 Oct 2008, Greg KH wrote:
>
> Here is the big batch of USB patches for your 2.6-git tree.

I really wish I were to get the occasional _clean_ pull for a change.

f*ck me, but why do people keep adding more and more warnings, and 
apparently not even bother to write clean code? The merge window is no 
excuse for badly written crap.

  drivers/usb/host/ehci-hcd.c: In function ?ehci_hcd_init?:
  drivers/usb/host/ehci-hcd.c:1115: warning: label ?err_debug? defined but not used

That particular function is just about the ugliest thing I've ever seen, 
and now it's adding annoying warnings too!

Yeah, I'm grumpy. The quality control during this merge window has been 
absolutely disgusting. I feel like I have to complain about every other 
pull I do, because people feel like another new warning isn't a problem. 
And every single time, the new warning is due to some total crap code.

		Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/