You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by jb...@apache.org on 2007/03/11 16:08:41 UTC

svn commit: r516918 - in /incubator/tuscany/java/sca/kernel: core/src/main/java/org/apache/tuscany/core/component/ core/src/main/java/org/apache/tuscany/core/implementation/ spi/src/main/java/org/apache/tuscany/spi/component/ spi/src/main/java/org/apac...

Author: jboynes
Date: Sun Mar 11 08:08:41 2007
New Revision: 516918

URL: http://svn.apache.org/viewvc?view=rev&rev=516918
Log:
remove obsolete isDestroyable method

Modified:
    incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/component/SystemSingletonAtomicComponent.java
    incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/implementation/PojoAtomicComponent.java
    incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/component/AtomicComponent.java
    incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/extension/AtomicComponentExtension.java

Modified: incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/component/SystemSingletonAtomicComponent.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/component/SystemSingletonAtomicComponent.java?view=diff&rev=516918&r1=516917&r2=516918
==============================================================================
--- incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/component/SystemSingletonAtomicComponent.java (original)
+++ incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/component/SystemSingletonAtomicComponent.java Sun Mar 11 08:08:41 2007
@@ -68,10 +68,6 @@
         return false;
     }
 
-    public boolean isDestroyable() {
-        return false;
-    }
-
     public int getInitLevel() {
         return 0;
     }

Modified: incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/implementation/PojoAtomicComponent.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/implementation/PojoAtomicComponent.java?view=diff&rev=516918&r1=516917&r2=516918
==============================================================================
--- incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/implementation/PojoAtomicComponent.java (original)
+++ incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/implementation/PojoAtomicComponent.java Sun Mar 11 08:08:41 2007
@@ -112,10 +112,6 @@
         componentContext = new ComponentContextImpl(this);
     }
 
-    public boolean isDestroyable() {
-        return destroyInvoker != null;
-    }
-
     public void destroy(Object instance) throws TargetDestructionException {
         if (destroyInvoker != null) {
             try {
@@ -129,7 +125,7 @@
     public boolean isOptimizable() {
         // stateless implementations that require a destroy callback cannot be optimized since the callback is
         // performed by the JavaTargetInvoker
-        return !(getScope() == Scope.STATELESS && isDestroyable());
+        return !(getScope() == Scope.STATELESS && destroyInvoker != null);
     }
 
     public Object getTargetInstance() throws TargetResolutionException {

Modified: incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/component/AtomicComponent.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/component/AtomicComponent.java?view=diff&rev=516918&r1=516917&r2=516918
==============================================================================
--- incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/component/AtomicComponent.java (original)
+++ incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/component/AtomicComponent.java Sun Mar 11 08:08:41 2007
@@ -35,13 +35,6 @@
     boolean isEagerInit();
 
     /**
-     * Returns true if component instances receive destroy events.
-     *
-     * @return true if component instances receive destroy events
-     */
-    boolean isDestroyable();
-
-    /**
      * Returns the initialization level for this component.
      *
      * @return the initialization level for this component

Modified: incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/extension/AtomicComponentExtension.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/extension/AtomicComponentExtension.java?view=diff&rev=516918&r1=516917&r2=516918
==============================================================================
--- incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/extension/AtomicComponentExtension.java (original)
+++ incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/extension/AtomicComponentExtension.java Sun Mar 11 08:08:41 2007
@@ -73,10 +73,6 @@
         return initLevel > 0;
     }
 
-    public boolean isDestroyable() {
-        return false;
-    }
-
     public long getMaxIdleTime() {
         return maxIdleTime;
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
For additional commands, e-mail: tuscany-commits-help@ws.apache.org