Mailing-List: contact derby-dev-help@db.apache.org; run by ezmlm
Precedence: bulk
Reply-To: <derby-dev@db.apache.org>
Received-SPF: neutral (asf.osuosl.org: local policy)
Message-ID: <44E264C4.8060608@sbcglobal.net>
Date: Tue, 15 Aug 2006 17:20:20 -0700
From: Mike Matrigali <mikem_app@sbcglobal.net>
Reply-To: mikem_app@sbcglobal.net
User-Agent: Mozilla Thunderbird 1.0 (Windows/20041206)
MIME-Version: 1.0
To: derby-dev@db.apache.org
Subject: Re: [VOTE]  Approve coding conventions for the Derby project
References: <44DD185F.8010808@sbcglobal.net>
In-Reply-To: <44DD185F.8010808@sbcglobal.net>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

my vote:-0

I continue to struggle with this issue.  I don't think we should
reformat the style of all the existing code, it is going to cause
merge hell.  But
if we don't why vote yes for this proposal as then it is just a lie
saying " Derby uses the "Code Conventions for the Java Programming 
Language" (http://java.sun.com/docs/codeconv/html/CodeConvTOC.doc.html) 
... ".  So I am unclear on what I am voting on.  Is it just the
following code convention is a perfectly acceptable way to submit
derby code, as well there might be others?

I haven't voted -1 as I don't have a better suggestion for a standard,
other than the code in a file should be clear and consistent.  Even
things I particularly don't like, like 80+ character lines are sometimes
more clear than hoops to make them less than 80 character.

I do think the space/tab issue causes headaches.  We could just document
it, but I actually think we should reformat the code with respect to
tabstop and just move to spaces.  I wish we were attacking the root
problems causing people pain up front.  What I see as problems are:
1) space/tab issue is causing problems.  Some tools just don't like 4 
space tabs.
2) do not make review/commit job harder by doing format changes along 
with "real" code changes.


Kathey Marsden wrote:
> This is a vote to define the coding conventions for the Derby project 
> per the db project guidelines http://db.apache.org/source.html
> Vote closes  10:00am, Wednesday, August  15.
> 
> [+1]  Adopt the coding convention described.
> [-1 ] Do not adopt the coding convention described.
> 
> The conventions outlined below will be published on the wiki page 
> http://wiki.apache.org/db-derby/DerbyContributorChecklist
> 
> Derby uses the "Code Conventions for the Java Programming Language"
> (http://java.sun.com/docs/codeconv/html/CodeConvTOC.doc.html) with these
> amendments:
> - space indentation (no tabs).
> - Derby does not discourage deferring variable declaration to the first 
> use.
> - Lines should be limited to 80 characters
> - @author tags should not be used at all
> 
> Note: There is a great deal of existing code that does not match this
> convention. Changes to existing code should match the surrounding code
> for readability, matching tabs or spaces as appropriate (see Tabs) .
> Patches should not have white space diffs. Code and diffs should be
> readable in  context.
> 
> 
> 
> 
>