avalon-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mcconn...@apache.org
Subject cvs commit: avalon-sandbox/merlin/src/java/org/apache/avalon/merlin/container/impl DefaultContainer.java
Date Mon, 17 Feb 2003 12:13:15 GMT
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: cvs-unsubscribe@avalon.apache.org
For additional commands, e-mail: cvs-help@avalon.apache.org


Mime
View raw message