From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:39690) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QyqCp-0001h5-FC for qemu-devel@nongnu.org; Wed, 31 Aug 2011 15:12:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QyqCn-0001Cn-U0 for qemu-devel@nongnu.org; Wed, 31 Aug 2011 15:12:11 -0400 Received: from mail-ew0-f45.google.com ([209.85.215.45]:47885) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QyqCn-0001Cg-N5 for qemu-devel@nongnu.org; Wed, 31 Aug 2011 15:12:09 -0400 Received: by ewy24 with SMTP id 24so766421ewy.4 for ; Wed, 31 Aug 2011 12:12:08 -0700 (PDT) Date: Wed, 31 Aug 2011 21:12:06 +0200 From: "Edgar E. Iglesias" Message-ID: <20110831191206.GD17959@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> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4E5E8202.1040109@codemonkey.ws> 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 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. > Is the next argument going to be that every subsystem should be able > to have its documentation in it's preferred natural language such > that the documentation for the block layer is in Esperanto? Heh, I think you missunderstood me. I'm talking about details that dont matter. Like, would you like to enforce American english or UK english? More that kind of question. > Consistent coding style makes the tree a single code base, instead > of a bunch of independent islands. This encourages sharing code and > ideas across subsystems. Too often, we reproduce the same thing and > over again in different subsystems (and even different machine > architectures). I dont think the latter has to do with codingstyle consistensy. But I agree with the former. Cheers