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]

Reply via email to