donaldp     2002/10/06 23:32:06

  Modified:    extension/src/java/org/apache/avalon/excalibur/packagemanager/impl
                        DelegatingExtensionManager.java
  Log:
  Updated method names to reflect new classnames
  
  Revision  Changes    Path
  1.4       +41 -17    
jakarta-avalon-excalibur/extension/src/java/org/apache/avalon/excalibur/packagemanager/impl/DelegatingExtensionManager.java
  
  Index: DelegatingExtensionManager.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-avalon-excalibur/extension/src/java/org/apache/avalon/excalibur/packagemanager/impl/DelegatingExtensionManager.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- DelegatingExtensionManager.java   2 Oct 2002 01:52:20 -0000       1.3
  +++ DelegatingExtensionManager.java   7 Oct 2002 06:32:06 -0000       1.4
  @@ -13,7 +13,7 @@
   import org.apache.avalon.excalibur.packagemanager.OptionalPackage;
   
   /**
  - * A {@link org.apache.avalon.excalibur.packagemanager.ExtensionManager} that can 
delegate to multiple
  + * A {@link ExtensionManager} that can delegate to multiple
    * different package repositories.
    *
    * @author <a href="mailto:peter at apache.org">Peter Donald</a>
  @@ -23,9 +23,9 @@
       implements ExtensionManager
   {
       /**
  -     * The list containing the package repositories.
  +     * The list containing the {@link ExtensionManager} objects.
        */
  -    private final ArrayList m_repositories = new ArrayList();
  +    private final ArrayList m_extensionManagers = new ArrayList();
   
       /**
        * Default constructor that does not add any repositories.
  @@ -35,39 +35,63 @@
       }
   
       /**
  -     * Default constructor that delegates to specified repositories.
  +     * Default constructor that delegates to specified extensionManagers.
        */
  -    public DelegatingExtensionManager( final ExtensionManager[] repositories )
  +    public DelegatingExtensionManager( final ExtensionManager[] extensionManagers )
       {
  -        for( int i = 0; i < repositories.length; i++ )
  +        for( int i = 0; i < extensionManagers.length; i++ )
           {
  -            addPackageRepository( repositories[ i ] );
  +            addExtensionManager( extensionManagers[ i ] );
           }
       }
   
       /**
  -     * Add a repository to list of repositories delegated to
  +     * Add a extensionManager to list of repositories delegated to
        * to find Optional Packages.
        *
  -     * @param repository the repository to add
  +     * @param extensionManager the extensionManager to add
        */
  -    protected synchronized void addPackageRepository( final ExtensionManager 
repository )
  +    protected synchronized void addExtensionManager( final ExtensionManager 
extensionManager )
       {
  -        if( !m_repositories.contains( repository ) )
  +        if( !m_extensionManagers.contains( extensionManager ) )
           {
  -            m_repositories.add( repository );
  +            m_extensionManagers.add( extensionManager );
           }
       }
   
       /**
  +     * Add a extensionManager to list of repositories delegated to
  +     * to find Optional Packages.
  +     *
  +     * @param extensionManager the extensionManager to add
  +     * @deprecated Use addExtensionManager instead
  +     */
  +    protected void addPackageRepository( final ExtensionManager extensionManager )
  +    {
  +        addExtensionManager( extensionManager );
  +    }
  +
  +    /**
        * Remove a repository from list of repositories delegated to
        * to find Optional Packages.
        *
        * @param repository the repository to remove
        */
  -    protected synchronized void removePackageRepository( final ExtensionManager 
repository )
  +    protected synchronized void removeExtensionManager( final ExtensionManager 
repository )
  +    {
  +        m_extensionManagers.remove( repository );
  +    }
  +
  +    /**
  +     * Remove a extensionManager from list of repositories delegated to
  +     * to find Optional Packages.
  +     *
  +     * @param extensionManager the extensionManager to remove
  +     * @deprecated Use removeExtensionManager instead.
  +     */
  +    protected void removePackageRepository( final ExtensionManager extensionManager 
)
       {
  -        m_repositories.remove( repository );
  +        removeExtensionManager( extensionManager );
       }
   
       /**
  @@ -81,11 +105,11 @@
       {
           final ArrayList resultPackages = new ArrayList();
   
  -        final int size = m_repositories.size();
  +        final int size = m_extensionManagers.size();
           for( int i = 0; i < size; i++ )
           {
               final ExtensionManager repository =
  -                (ExtensionManager)m_repositories.get( i );
  +                (ExtensionManager)m_extensionManagers.get( i );
               final OptionalPackage[] packages =
                   repository.getOptionalPackages( extension );
               if( null == packages || 0 == packages.length )
  
  
  

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to