avalon-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From l...@apache.org
Subject cvs commit: jakarta-avalon-excalibur/component/src/java/org/apache/avalon/excalibur/component DefaultComponentHandler.java
Date Wed, 21 Aug 2002 05:04:36 GMT
leif        2002/08/20 22:04:36

  Modified:    component/src/java/org/apache/avalon/excalibur/component
                        DefaultComponentHandler.java
  Log:
  Checkstyle fixes.
  
  Revision  Changes    Path
  1.3       +31 -8     jakarta-avalon-excalibur/component/src/java/org/apache/avalon/excalibur/component/DefaultComponentHandler.java
  
  Index: DefaultComponentHandler.java
  ===================================================================
  RCS file: /home/cvs/jakarta-avalon-excalibur/component/src/java/org/apache/avalon/excalibur/component/DefaultComponentHandler.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- DefaultComponentHandler.java	2 Jun 2002 06:03:01 -0000	1.2
  +++ DefaultComponentHandler.java	21 Aug 2002 05:04:36 -0000	1.3
  @@ -39,6 +39,17 @@
        * Create a ComponentHandler that takes care of hiding the details of
        * whether a Component is ThreadSafe, Poolable, or SingleThreaded.
        * It falls back to SingleThreaded if not specified.
  +     *
  +     * @param componentClass Class of the component of the handler being
  +     *                       created.
  +     * @param config The configuration for the component.
  +     * @param manager The ComponentManager which will be managing the
  +     *                Component.
  +     * @param context The current context.
  +     * @param roles The current RoleManager.
  +     * @param logkit The current LogKitLoggerManager.
  +     *
  +     * @throws Exception If there are any problems creating the handler.
        */
       protected DefaultComponentHandler( final Class componentClass,
                                          final Configuration config,
  @@ -60,6 +71,8 @@
        * @param factory The factory object which is responsible for creating the components
        *                managed by the ComponentHandler.
        * @param config The configuration to use to configure the pool.
  +     *
  +     * @throws Exception If there are any problems creating the handler.
        */
       public DefaultComponentHandler( final DefaultComponentFactory factory,
                                       final Configuration config )
  @@ -82,20 +95,27 @@
   
           if( getLogger().isDebugEnabled() )
           {
  -            getLogger().debug( "ComponentHandler initialized for: " + m_factory.getCreatedClass().getName()
);
  +            getLogger().debug( "ComponentHandler initialized for: "
  +                + m_factory.getCreatedClass().getName() );
           }
           m_initialized = true;
       }
   
       /**
        * Get a reference of the desired Component
  +     *
  +     * @return A component instance.
  +     *
  +     * @throws Exception If there are any problems encountered acquiring a
  +     *                   component instance.
        */
       protected Component doGet()
           throws Exception
       {
           if( !m_initialized )
           {
  -            throw new IllegalStateException( "You cannot get a component from an uninitialized
holder." );
  +            throw new IllegalStateException(
  +                "You cannot get a component from an uninitialized holder." );
           }
   
           if( m_disposed )
  @@ -108,12 +128,15 @@
   
       /**
        * Return a reference of the desired Component
  +     *
  +     * @param component Component to be be put/released back to the handler.
        */
       protected void doPut( final Component component )
       {
           if( !m_initialized )
           {
  -            throw new IllegalStateException( "You cannot put a component in an uninitialized
holder." );
  +            throw new IllegalStateException(
  +                "You cannot put a component in an uninitialized holder." );
           }
   
           try
  @@ -124,8 +147,8 @@
           {
               if( getLogger().isWarnEnabled() )
               {
  -                getLogger().warn( "Error decommissioning component: " +
  -                                  m_factory.getCreatedClass().getName(), e );
  +                getLogger().warn( "Error decommissioning component: "
  +                    + m_factory.getCreatedClass().getName(), e );
               }
           }
       }
  @@ -148,8 +171,8 @@
           {
               if( getLogger().isWarnEnabled() )
               {
  -                getLogger().warn( "Error decommissioning component: " +
  -                                  m_factory.getCreatedClass().getName(), e );
  +                getLogger().warn( "Error decommissioning component: "
  +                    + m_factory.getCreatedClass().getName(), e );
               }
           }
   
  
  
  

--
To unsubscribe, e-mail:   <mailto:avalon-cvs-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:avalon-cvs-help@jakarta.apache.org>


Mime
View raw message