From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:56657) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QyqJF-0004Vd-D8 for qemu-devel@nongnu.org; Wed, 31 Aug 2011 15:18:50 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QyqJE-0002bT-BU for qemu-devel@nongnu.org; Wed, 31 Aug 2011 15:18:49 -0400 Received: from mail-ew0-f45.google.com ([209.85.215.45]:57966) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QyqJD-0002bN-Vt for qemu-devel@nongnu.org; Wed, 31 Aug 2011 15:18:48 -0400 Received: by ewy24 with SMTP id 24so772481ewy.4 for ; Wed, 31 Aug 2011 12:18:47 -0700 (PDT) Date: Wed, 31 Aug 2011 21:18:44 +0200 From: "Edgar E. Iglesias" Message-ID: <20110831191844.GA18854@zapo> References: <4E5E3789.3010603@codemonkey.ws> <4E5E39DF.3080609@codemonkey.ws> <4E5E4540.5030408@codemonkey.ws> <4E5E5BF3.50801@codemonkey.ws> <20110831175934.GA17959@zapo> <4E5E8202.1040109@codemonkey.ws> <20110831191206.GD17959@zapo> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20110831191206.GD17959@zapo> Subject: Re: [Qemu-devel] [PATCH] Add support for r6040 NIC List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: qemu-devel@nongnu.org, bifferos On Wed, Aug 31, 2011 at 09:12:06PM +0200, Edgar E. Iglesias wrote: > On Wed, Aug 31, 2011 at 01:48:34PM -0500, Anthony Liguori wrote: > > On 08/31/2011 12:59 PM, Edgar E. Iglesias wrote: > > >On Wed, Aug 31, 2011 at 11:06:11AM -0500, Anthony Liguori wrote: > > >>On 08/31/2011 09:35 AM, malc wrote: > > >>>On Wed, 31 Aug 2011, Anthony Liguori wrote: > > >>> > > >>>>Upper case field names are not okay. If you think coding style isn't clear, > > >>>>that's a bug in coding style. > > >>> > > >>>Sez hu? Coding style is garbage that should be thrown out of the window. > > >>>As for looking, yeah, i'm looking at usb with it's lovely hungarian > > >>>fields, should we stampede to "fix" it? > > >>> > > >>>If the one who's going to maintain the code is fine with whatever naming > > >>>is used so be it. > > >> > > >>No. That's how we got into the coding style mess we're in in the > > >>first place. > > > > > >TBH, the codingstyle in QEMU is the least of "problems" we are facing. > > >We've got lack of documentation, lack of tests, lack of contributors, > > >etc, etc. IMO, those bring codingstyle issues into the pretty much > > >neglectable space. > > > > I don't think we lack contributors. Documentation and tests are > > really about discipline. If we can't even be bothered to maintain > > consistency in variable naming, do you really expected that we can > > be disciplined in writing documentation and tests? > > Yes I do. It's not white and black, it's not about making the code > completely inconsistent or 100 consistent. It's about find a level > of consistency that is acceptable and doesn't cost too much to > maintain. Now if that's my opinion, then the 99999 dollar question is: why am I wasting so much time on discusing it? And btw Bifferos, don't worry. Not all patches you contribute will cause this much controversy :) Cheers