You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@avalon.apache.org by do...@apache.org on 2002/05/14 14:41:13 UTC

cvs commit: jakarta-avalon-phoenix/src/java/org/apache/avalon/phoenix/components/lifecycle ComponentEntry.java

donaldp     02/05/14 05:41:13

  Modified:    src/java/org/apache/avalon/phoenix/components/lifecycle
                        ComponentEntry.java
  Log:
  Made state mutation done via final methods.
  Also made invalidate set variables to null directly rather than via methods (which could cause recursion).
  
  Revision  Changes    Path
  1.2       +4 -4      jakarta-avalon-phoenix/src/java/org/apache/avalon/phoenix/components/lifecycle/ComponentEntry.java
  
  Index: ComponentEntry.java
  ===================================================================
  RCS file: /home/cvs/jakarta-avalon-phoenix/src/java/org/apache/avalon/phoenix/components/lifecycle/ComponentEntry.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- ComponentEntry.java	14 May 2002 11:58:54 -0000	1.1
  +++ ComponentEntry.java	14 May 2002 12:41:13 -0000	1.2
  @@ -29,12 +29,12 @@
           return m_name;
       }
   
  -    public synchronized State getState()
  +    public final synchronized State getState()
       {
           return m_state;
       }
   
  -    public synchronized void setState( final State state )
  +    public final synchronized void setState( final State state )
       {
           m_state = state;
       }
  @@ -51,7 +51,7 @@
   
       public synchronized void invalidate()
       {
  -        setObject( null );
  -        setState( State.VOID );
  +        m_state = State.VOID;
  +        m_object = null;
       }
   }
  
  
  

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>