Ok, 

Ilkka Priha's code is now in CVS. It is *beautiful* and much needed.

His Factory/Pool Service is *way* cool (and probably a duplication of what
is in Avalon...oh well...one day we will all work together...).

I made the following changes to the code he sent me...

p.s. Ilkka, I found your mug shot^H^H^H^H^H^H^H^Hpicture...
         <http://www.simsoft.fi/images/IlkkaPriha.jpg>

:-)

-jon


Changes:

added @version tags to files that were missing it.

removed double check locking code in
TurbineFactoryService.getPrimitiveClass()

changed TFS/TPS to init() instead of init(ServletConfig)...don't init until
needed...

removed tabs

fixed 2 space code with 4 space code

TurbineFactoryService.init() now uses props.getVector() instead of
attempting to re-parse the string...

fixed these compile errors:

[javac] 
/Users/jon/checkout/turbine/bin/src/org/apache/turbine/services/factory/Turb
ineFactoryService.java:153: Reference to SERVICE_NAME is ambiguous. It is
defined in interface org.apache.turbine.services.factory.FactoryService and
interface org.apache.turbine.services.Service.
    [javac]                         SERVICE_NAME + ": No such class loader "
+ loader,x);
    [javac]                         ^
    [javac] 
/Users/jon/checkout/turbine/bin/src/org/apache/turbine/services/pool/Turbine
PoolService.java:134: Reference to SERVICE_NAME is ambiguous. It is defined
in interface org.apache.turbine.services.factory.FactoryService and
interface org.apache.turbine.services.Service.
    [javac]                         SERVICE_NAME + ": Incorrect pool
capacity " + str,x);

thanks,

-jon

-- 
If you come from a Perl or PHP background, JSP is a way to take
your pain to new levels. --Anonymous
<http://jakarta.apache.org/velocity/> && <http://java.apache.org/turbine/>



------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?:           [EMAIL PROTECTED]

Reply via email to