Return-Path: Delivered-To: apmail-openjpa-dev-archive@www.apache.org Received: (qmail 77128 invoked from network); 5 Jun 2009 14:33:48 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 5 Jun 2009 14:33:48 -0000 Received: (qmail 97717 invoked by uid 500); 5 Jun 2009 14:34:00 -0000 Delivered-To: apmail-openjpa-dev-archive@openjpa.apache.org Received: (qmail 97661 invoked by uid 500); 5 Jun 2009 14:33:59 -0000 Mailing-List: contact dev-help@openjpa.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@openjpa.apache.org Delivered-To: mailing list dev@openjpa.apache.org Received: (qmail 97651 invoked by uid 99); 5 Jun 2009 14:33:59 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 05 Jun 2009 14:33:59 +0000 X-ASF-Spam-Status: No, hits=-0.0 required=10.0 tests=SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of lists+1214986235816-210739@n2.nabble.com designates 216.139.236.158 as permitted sender) Received: from [216.139.236.158] (HELO kuber.nabble.com) (216.139.236.158) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 05 Jun 2009 14:33:50 +0000 Received: from tervel.nabble.com ([192.168.236.150]) by kuber.nabble.com with esmtp (Exim 4.63) (envelope-from ) id 1MCaU5-0007Wa-UM for dev@openjpa.apache.org; Fri, 05 Jun 2009 07:33:29 -0700 Message-ID: <1244212409935-3030641.post@n2.nabble.com> Date: Fri, 5 Jun 2009 07:33:29 -0700 (PDT) From: Rick Curtis To: dev@openjpa.apache.org Subject: Re: 80-column line width & auto-generated model classes In-Reply-To: <89c0c52c0906050727j604215e5s19ce26d2ec2a0e50@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Nabble-From: curtisr7@gmail.com References: <1244133496189-3025746.post@n2.nabble.com> <915B5A2C-186E-4CD8-B569-C248FF35C795@SUN.com> <1244208989974-3030320.post@n2.nabble.com> <72c1350f0906050643g4ff8f1acs59badcd93a86a782@mail.gmail.com> <89c0c52c0906050727j604215e5s19ce26d2ec2a0e50@mail.gmail.com> X-Virus-Checked: Checked by ClamAV on apache.org +1 I think that we need to enforce some sort of coding standards, but I don't think we should be blindly enforcing conventions that are out of date. Take a look at the 'official' Sun code conventions [1] and note that they haven't been updated since April 20, 1999. I particularly like the reason for the 80-column line [2]... 'Avoid lines longer than 80 characters, since they're not handled well by many terminals and tools.' [1] http://java.sun.com/docs/codeconv/ [2] http://java.sun.com/docs/codeconv/html/CodeConventions.doc3.html#313 Kevin Sutter wrote: > > A few interesting points with this discussion... > > 1) Under what conditions should we be checking in these generated > metamodel > classes? We don't want the metamodel classes generated for all of our > Entities, do we? Seems like a waste of build time. So, how are we > determining which Entities should have generated metamodel classes? > > 2) Our processing in this arena can help define a "best practice" for our > eventual JPA 2.0 users. That is, how and when to do the metamodel > generation. How to package these generated classes for the application. > And, whether these should be checked into SVN or not. > > 3) And, why do we continue to have the 80 column limit for our source > files? :-) It seems so archaic. > > Kevin > -- View this message in context: http://n2.nabble.com/80-column-line-width---auto-generated-model-classes-tp3025746p3030641.html Sent from the OpenJPA Developers mailing list archive at Nabble.com.