You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by jg...@apache.org on 2015/04/15 11:59:44 UTC

tomee git commit: Look this up lazily

Repository: tomee
Updated Branches:
  refs/heads/tomee-1.7.x 3f17e237d -> f85e65d12


Look this up lazily


Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/f85e65d1
Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/f85e65d1
Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/f85e65d1

Branch: refs/heads/tomee-1.7.x
Commit: f85e65d129d990914f6ca6e93cb03fb990c2a41e
Parents: 3f17e23
Author: Jonathan Gallimore <jo...@jrg.me.uk>
Authored: Wed Apr 15 10:59:32 2015 +0100
Committer: Jonathan Gallimore <jo...@jrg.me.uk>
Committed: Wed Apr 15 10:59:32 2015 +0100

----------------------------------------------------------------------
 .../java/org/apache/openejb/assembler/classic/Assembler.java     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tomee/blob/f85e65d1/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java
----------------------------------------------------------------------
diff --git a/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java b/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java
index 0e7373d..6ced0e9 100644
--- a/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java
+++ b/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java
@@ -1606,7 +1606,7 @@ public class Assembler extends AssemblerTool implements org.apache.openejb.spi.A
 
     private void destroyResource(final String name, final String className, final Object object) {
 
-        final Method preDestroy = findPreDestroy(object);
+        Method preDestroy = null;
 
         if (object instanceof ResourceAdapterReference) {
             final ResourceAdapterReference resourceAdapter = (ResourceAdapterReference) object;
@@ -1666,7 +1666,7 @@ public class Assembler extends AssemblerTool implements org.apache.openejb.spi.A
             } catch (final RuntimeException e) {
                 logger.error(e.getMessage(), e);
             }
-        } else if (preDestroy != null) {
+        } else if ((preDestroy = findPreDestroy(object)) != null) {
             logger.debug("Calling @PreDestroy on: " + className);
             try {
                 preDestroy.invoke(object);