From java-dev-return-19016-apmail-lucene-java-dev-archive=lucene.apache.org@lucene.apache.org Mon Mar 26 18:07:15 2007 Return-Path: Delivered-To: apmail-lucene-java-dev-archive@www.apache.org Received: (qmail 76734 invoked from network); 26 Mar 2007 18:07:13 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 26 Mar 2007 18:07:13 -0000 Received: (qmail 83724 invoked by uid 500); 26 Mar 2007 18:07:17 -0000 Delivered-To: apmail-lucene-java-dev-archive@lucene.apache.org Received: (qmail 83679 invoked by uid 500); 26 Mar 2007 18:07:17 -0000 Mailing-List: contact java-dev-help@lucene.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: java-dev@lucene.apache.org Delivered-To: mailing list java-dev@lucene.apache.org Received: (qmail 83668 invoked by uid 99); 26 Mar 2007 18:07:17 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 26 Mar 2007 11:07:17 -0700 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received-SPF: neutral (herse.apache.org: local policy) Received: from [204.127.192.84] (HELO rwcrmhc14.comcast.net) (204.127.192.84) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 26 Mar 2007 11:07:09 -0700 Received: from [192.168.168.15] (c-71-202-24-246.hsd1.ca.comcast.net[71.202.24.246]) by comcast.net (rwcrmhc14) with ESMTP id <20070326180648m14003ko6ae>; Mon, 26 Mar 2007 18:06:48 +0000 Message-ID: <46080BB8.1050604@apache.org> Date: Mon, 26 Mar 2007 11:06:48 -0700 From: Doug Cutting User-Agent: Thunderbird 1.5.0.10 (X11/20070306) MIME-Version: 1.0 To: java-dev@lucene.apache.org Subject: Re: whitespace References: In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org Yonik Seeley wrote: > The java coding standards do say > "Spaces after keywords but no spaces either before or after > parentheses in method calls" > > "if (a)" and "foo(a)" Lucene's formatting guidelines are stated at: http://wiki.apache.org/lucene-java/HowToContribute In short, they say: Sun's conventions, *except* two-space indentation. I think it is fair to bounce a patch for flagrantly violating these, or to unilaterally repair its formatting prior to commit. It's painful to re-format code after it is committed. That's the current policy. It could be changed with a vote. Doug --------------------------------------------------------------------- To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org For additional commands, e-mail: java-dev-help@lucene.apache.org