From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:43550) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QyqNV-0006AY-QT for qemu-devel@nongnu.org; Wed, 31 Aug 2011 15:23:15 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QyqNT-0003n5-VT for qemu-devel@nongnu.org; Wed, 31 Aug 2011 15:23:13 -0400 Received: from mail-yi0-f45.google.com ([209.85.218.45]:48113) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QyqNT-0003my-Os for qemu-devel@nongnu.org; Wed, 31 Aug 2011 15:23:11 -0400 Received: by yih10 with SMTP id 10so958687yih.4 for ; Wed, 31 Aug 2011 12:23:11 -0700 (PDT) Message-ID: <4E5E8A1D.6020103@codemonkey.ws> Date: Wed, 31 Aug 2011 14:23:09 -0500 From: Anthony Liguori MIME-Version: 1.0 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> In-Reply-To: <20110831191206.GD17959@zapo> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] Add support for r6040 NIC List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Edgar E. Iglesias" Cc: qemu-devel@nongnu.org, bifferos On 08/31/2011 02:12 PM, Edgar E. Iglesias wrote: > On Wed, Aug 31, 2011 at 01:48:34PM -0500, Anthony Liguori wrote: >>> 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. I actually agree. I don't like the idea of absolutely enforcing a coding style that demands no white space at the end of a line (if you can't see it, why in the world would you care?). But coding style deviations that make the code look foreign, like using CamelCase for field names, seems important to me. And I respect that other things seem important to other people (even invisible things like trailing white space). So even though I wouldn't want to reject a patch because of coding style in some cases, I think it's important that we do our best to enforce it. Regards, Anthony Liguori