tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Remy Maucherat <r...@apache.org>
Subject Re: svn commit: r542189 [1/2] - in /tomcat/trunk/java/org/apache: ./ catalina/core/ catalina/deploy/ catalina/security/ catalina/startup/ catalina/util/ jasper/ jasper/compiler/ jasper/runtime/ jasper/servlet/
Date Mon, 28 May 2007 11:46:04 GMT
Filip Hanik - Dev Lists wrote:
> Remy Maucherat wrote:
>> Filip Hanik - Dev Lists wrote:
>>> we're sticking with the 'org.apache' package name?
>>
>> Yes.
>>
>>> wouldn't this branch be a good way to fix that?
>>
>> Nice try.
> I wouldn't feel like I contributed if I didn't ;)

The main problem is that this solution is bad, but the other solutions 
are worse.

 > +        checkWorkerThread();
 > +        if (getEventType()!=EventType.BEGIN) {
 > +            throw new IllegalStateException("Configure can only be 
called during the BEGIN event.");
 > +        }

This stuff really needs to go ;)

Rémy

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


Mime
View raw message