Return-Path: 
 <log4j-dev-return-3350-qmlist-jakarta-archive-log4j-dev=nagoya.apache.org@jakarta.apache.org>
Delivered-To: apmail-jakarta-log4j-dev-archive@apache.org
Received: (qmail 55719 invoked from network); 18 Feb 2003 18:58:13 -0000
Received: from exchange.sun.com (192.18.33.10)
  by daedalus.apache.org with SMTP; 18 Feb 2003 18:58:13 -0000
Received: (qmail 16924 invoked by uid 97); 18 Feb 2003 18:59:49 -0000
Delivered-To: qmlist-jakarta-archive-log4j-dev@nagoya.betaversion.org
Received: (qmail 16917 invoked from network); 18 Feb 2003 18:59:49 -0000
Received: from daedalus.apache.org (HELO apache.org) (208.185.179.12)
  by nagoya.betaversion.org with SMTP; 18 Feb 2003 18:59:49 -0000
Received: (qmail 54756 invoked by uid 500); 18 Feb 2003 18:58:03 -0000
Mailing-List: contact log4j-dev-help@jakarta.apache.org; run by ezmlm
Precedence: bulk
List-Unsubscribe: <mailto:log4j-dev-unsubscribe@jakarta.apache.org>
List-Subscribe: <mailto:log4j-dev-subscribe@jakarta.apache.org>
List-Help: <mailto:log4j-dev-help@jakarta.apache.org>
List-Post: <mailto:log4j-dev@jakarta.apache.org>
List-Id: "Log4J Developers List" <log4j-dev.jakarta.apache.org>
Reply-To: "Log4J Developers List" <log4j-dev@jakarta.apache.org>
Delivered-To: mailing list log4j-dev@jakarta.apache.org
Received: (qmail 54626 invoked from network); 18 Feb 2003 18:58:01 -0000
Received: from ns1.syntegra.com (150.143.16.2)
  by daedalus.apache.org with SMTP; 18 Feb 2003 18:58:01 -0000
Received: from [129.179.161.11] by ns1.syntegra.com with ESMTP for
 log4j-dev@jakarta.apache.org; Tue, 18 Feb 2003 18:58:05 Z
Received: from ns1.syntegra.com by cdsms.cdc.com with ESMTP for
 log4j-dev@jakarta.apache.org; Tue, 18 Feb 2003 12:58:04 -0600
Received: from cis-exchange.jedi.local by ns1.syntegra.com with ESMTP for
 log4j-dev@jakarta.apache.org; Tue, 18 Feb 2003 18:58:04 Z
Received: from us613005.ah.us.intra.syntegra.com ([129.179.77.171] unverified)
 by cis-exchange.jedi.local with Microsoft SMTPSVC(5.0.2195.5329);
	 Tue, 18 Feb 2003 12:58:03 -0600
Date: Tue, 18 Feb 2003 12:58:03 -0600
From: Jacob Kjome <hoju@visi.com>
X-Mailer: The Bat! (v1.54/10) Personal
Reply-To: Jacob Kjome <hoju@visi.com>
Organization: Springfield Nuclear Plant
X-Priority: 3 (Normal)
Message-Id: <15194421701.20030218125803@visi.com>
To: "Log4J Developers List" <log4j-dev@jakarta.apache.org>
Subject: Re[2]: Indentation!!!
In-Reply-To: <F73733CC5E11D41189E700D0B74A0C9E0529F5E7@exchange.bevocal.com>
References: <F73733CC5E11D41189E700D0B74A0C9E0529F5E7@exchange.bevocal.com>
X-OriginalArrivalTime: 18 Feb 2003 18:58:03.0960 (UTC)
 FILETIME=[AA68AB80:01C2D77F]
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N
X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N

Hello Mark,

In addition to making mods to build.xml, selector and servlet
initializers, I also took the liberty of adding the mods to the
sunCodingConvention.xml that we agreed to.  So, make sure it isn't
already there before you check anything in.

Jake

Tuesday, February 18, 2003, 12:25:47 PM, you wrote:

MW> I didn't get home from work until after midnight, so needless to say, I did
MW> not check in the changes as promised.  I will do it this evening though.

MW> -Mark

>> -----Original Message-----
>> From: Mark Womack 
>> Sent: Monday, February 17, 2003 12:01 PM
>> To: 'Log4J Developers List'
>> Subject: RE: Indentation!!!
>> 
>> 
>> Jake,
>>  
>> Thanks.  I'll check in the patch later tonight.
>>  
>> Did you run into any other discrepencies between what jalopy 
>> changes and
>> checkstyle checks?
>>  
>> -Mark
>> 
>> -----Original Message-----
>> From: Jacob Kjome [mailto:hoju@visi.com]
>> Sent: Sunday, February 16, 2003 8:14 PM
>> To: Log4J Developers List
>> Subject: RE: Indentation!!!
>> 
>> 
>> 
>> One thing that isn't being enforced is the order of keyword 
>> modifiers.  The
>> following change would enable that.
>> 
>> change
>>             <modifier>
>>  
>> <order>public|protected|private|abstract|static|final|synchron
>> ized|transient
>> |volatile|native|strictfp</order>
>>             </modifier>
>> to
>>             <modifier>
>>                 <enable>true</enable>
>>  
>> <order>public|protected|private|abstract|static|final|synchron
>> ized|transient
>> |volatile|native|strictfp</order>
>>             </modifier>
>> 
>> 
>> Otherwise, when you run Jalopy and your keywords are out of the order
>> specified above, they won't get corrected and checkstyle will 
>> still report
>> errors.
>> 
>> As a side note, I can't figure out how to make work with 
>> Jalopy is changing
>> single line comment lines to be 79 characters long or less.  
>> They remain
>> unmodified where multi-line comments get corrected.  Anyone 
>> know a way to do
>> that?  If not, then single line comments will always require
>> hand-modification to bring them into compliance.  Jalopy 
>> alone won't do the
>> trick.
>> 
>> 
>> Oh, one other thing...
>> 
>> I order to run the "editJalopyPreferences" target, I had to move the
>> following code from inside the "runJalopy" target to a place 
>> outside that
>> target so both targets could see it:
>> 
>>   <path id="jalopy.path">
>>     <fileset dir="${jalopy.lib}">
>>       <include name="*.jar" />
>>     </fileset>
>>   </path>
>> 
>> 
>> I also made the jalopy targets which weren't depending on 
>> "init" to depend
>> on init to be consistent with the other targets.  I'm 
>> attaching a diff of my
>> preferred changes against the current build.xml.
>> 
>> Jake
>> 
>> At 11:15 AM 2/15/2003 -0800, you wrote:
>> 
>> 
>> Done.  I changed "continuation" and "general" from 4 to 2.  
>> If there are any
>> other changes that should be applied, let me know.
>> 
>> -Mark
>> 
>> > -----Original Message-----
>> > From: Ceki Gulcu [ mailto:ceki@qos.ch <mailto:ceki@qos.ch> ]
>> > Sent: Saturday, February 15, 2003 7:38 AM
>> > To: log4j-dev@jakarta.apache.org
>> > Subject: Indentation!!!
>> >
>> >
>> >
>> > Mark,
>> >
>> > I'd just like to point out that although log4j generally 
>> adheres to Suns
>> > coding conventions, it uses 2 spaces for indentation and 
>> not 4. I would
>> > like to request that the file sunCodingConvention.xml be modified to
>> > reflect this difference. TIA,
>> >
>> >
>> > --
>> > Ceki
>> >
>> >
>> > 
>> ---------------------------------------------------------------------
>> > To unsubscribe, e-mail: log4j-dev-unsubscribe@jakarta.apache.org
>> > For additional commands, e-mail: log4j-dev-help@jakarta.apache.org
>> >
>> 
>> 
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: log4j-dev-unsubscribe@jakarta.apache.org
>> For additional commands, e-mail: log4j-dev-help@jakarta.apache.org
>> 
>> 

MW> ---------------------------------------------------------------------
MW> To unsubscribe, e-mail: log4j-dev-unsubscribe@jakarta.apache.org
MW> For additional commands, e-mail: log4j-dev-help@jakarta.apache.org



-- 
Best regards,
 Jacob                            mailto:hoju@visi.com


---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: log4j-dev-help@jakarta.apache.org