tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Costin Manolache <cmanola...@yahoo.com>
Subject Re: cvs commit: jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/core StandardWrapper.java
Date Wed, 06 Nov 2002 08:24:04 GMT
Remy Maucherat wrote:

> costin@apache.org wrote:
> 
>> costin      2002/11/05 13:35:00
>>
>>   Modified:    catalina/src/share/org/apache/catalina/core
>>                         StandardWrapper.java
>>   Log:
>>   Use commons-logging.
>>
>>   Add a short message for servlets that take a long time to load
>>   ( right now it's 200 ms - we can increase it, but no need to
>>   make it configurable, it's mostly do debug perf. ).
>>
>>   I used idea to edit the file - and it removed unused imports and
>>   forced me to fix some name mismatches... Probably should go back
>>   to emacs...
> 
> 
> Especially since it converted the imports to * imports (evil, evil).
> Please stop using it or disable the feature (and can you revert the
> changes on the imports ?).

I can revert the change - I just had it on my computer for a long
while, it's really not that important. I would like to slowly move
all the logging to c-l, but there is no urgency.

Regarding Idea - it warned me that about 10 imports
were not beeing used - and I let it 'fix' it. It seems it uses * if
more than 5 files in the same package are used.

Overall I'm really impressed with idea - I don't think I'll be able
to return to emacs. The code completion and runtime correction is
great, and the speed is decent. 

And with few fixes I was able to even run tomcat inside and debug
it... I still have 20 more days to evaluate it, but so far it seems
its worth the money. I'll just try once more with netbeans and eclipse.

BTW - what's evil in import * ? I never understood.

Costin



--
To unsubscribe, e-mail:   <mailto:tomcat-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:tomcat-dev-help@jakarta.apache.org>


Mime
View raw message