mcconnell 2003/02/17 04:13:15 Modified: merlin/src/java/org/apache/avalon/merlin/container/impl DefaultContainer.java Log: Identification and documentation of semantic conflict related to container startup/suspend/resume/shutdown. Revision Changes Path 1.5 +12 -1 avalon-sandbox/merlin/src/java/org/apache/avalon/merlin/container/impl/DefaultContainer.java Index: DefaultContainer.java =================================================================== RCS file: /home/cvs/avalon-sandbox/merlin/src/java/org/apache/avalon/merlin/container/impl/DefaultContainer.java,v retrieving revision 1.4 retrieving revision 1.5 diff -u -r1.4 -r1.5 --- DefaultContainer.java 12 Feb 2003 16:28:20 -0000 1.4 +++ DefaultContainer.java 17 Feb 2003 12:13:15 -0000 1.5 @@ -395,6 +395,17 @@ } try { + + // + // ## Problems in the following code: + // ## 1. we are assuming an Avalon startup strategy + // ## when in fact this should be handled by the appliance + // ## 2. this could be a singleton object that has already been + // ## started. + // ## Solution - remove the notion of container startup, + // ## suspension, resumption and stopping + // + Object object = appliance.resolve( this ); m_components.put( appliance, object ); if( !appliance.getStartupPolicy() )
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]