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:18:20 UTC
tomee git commit: TOMEE-1547 PMD/checkstyle
Repository: tomee
Updated Branches:
refs/heads/tomee-1.7.x d4aec8fae -> ad3b535e0
TOMEE-1547 PMD/checkstyle
Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/ad3b535e
Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/ad3b535e
Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/ad3b535e
Branch: refs/heads/tomee-1.7.x
Commit: ad3b535e082394cf464301423b0b8527409e3631
Parents: d4aec8f
Author: Jonathan Gallimore <jo...@jrg.me.uk>
Authored: Wed Apr 15 10:18:02 2015 +0100
Committer: Jonathan Gallimore <jo...@jrg.me.uk>
Committed: Wed Apr 15 10:18:02 2015 +0100
----------------------------------------------------------------------
.../apache/openejb/assembler/classic/Assembler.java | 14 +++++---------
.../org/apache/openejb/config/DeploymentLoader.java | 2 --
.../org/apache/openejb/config/ReadDescriptors.java | 1 +
3 files changed, 6 insertions(+), 11 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tomee/blob/ad3b535e/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 0667bd3..47930c0 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
@@ -65,7 +65,6 @@ import org.apache.openejb.config.ConfigurationFactory;
import org.apache.openejb.config.NewLoaderLogic;
import org.apache.openejb.config.QuickJarsTxtParser;
import org.apache.openejb.config.TldScanner;
-import org.apache.openejb.config.sys.Resource;
import org.apache.openejb.core.ConnectorReference;
import org.apache.openejb.core.CoreContainerSystem;
import org.apache.openejb.core.CoreUserTransaction;
@@ -79,7 +78,6 @@ import org.apache.openejb.core.ivm.IntraVmProxy;
import org.apache.openejb.core.ivm.naming.ContextualJndiReference;
import org.apache.openejb.core.ivm.naming.IvmContext;
import org.apache.openejb.core.ivm.naming.IvmJndiFactory;
-import org.apache.openejb.core.ivm.naming.JndiUrlReference;
import org.apache.openejb.core.ivm.naming.LazyObjectReference;
import org.apache.openejb.core.ivm.naming.Reference;
import org.apache.openejb.core.security.SecurityContextHandler;
@@ -134,7 +132,6 @@ import org.apache.openejb.util.classloader.ClassLoaderAwareHandler;
import org.apache.openejb.util.classloader.URLClassLoaderFirst;
import org.apache.openejb.util.proxy.ProxyFactory;
import org.apache.openejb.util.proxy.ProxyManager;
-import org.apache.webbeans.component.ResourceBean;
import org.apache.webbeans.config.WebBeansContext;
import org.apache.webbeans.inject.OWBInjector;
import org.apache.webbeans.logger.JULLoggerFactory;
@@ -198,7 +195,6 @@ import java.lang.annotation.Annotation;
import java.lang.instrument.ClassFileTransformer;
import java.lang.instrument.Instrumentation;
import java.lang.reflect.Constructor;
-import java.lang.reflect.InvocationTargetException;
import java.lang.reflect.Method;
import java.lang.reflect.Proxy;
import java.net.MalformedURLException;
@@ -968,7 +964,7 @@ public class Assembler extends AssemblerTool implements org.apache.openejb.spi.A
final Set<String> resourceIds = new HashSet<String>(appInfo.resourceIds);
final List<ResourceInfo> resourceList = config.facilities.resources;
- for (ResourceInfo resourceInfo : resourceList) {
+ for (final ResourceInfo resourceInfo : resourceList) {
if (!resourceIds.contains(resourceInfo.id)) {
continue;
}
@@ -976,7 +972,7 @@ public class Assembler extends AssemblerTool implements org.apache.openejb.spi.A
try {
final Class<?> cls = Class.forName(resourceInfo.className, true, classLoader);
final Method postConstruct = findMethodAnnotatedWith(PostConstruct.class, cls);
- boolean initialize = "true".equalsIgnoreCase(String.valueOf(resourceInfo.properties.remove("InitializeAfterDeployment")));
+ final boolean initialize = "true".equalsIgnoreCase(String.valueOf(resourceInfo.properties.remove("InitializeAfterDeployment")));
if (postConstruct != null || initialize) {
Object resource = containerSystemContext.lookup(OPENEJB_RESOURCE_JNDI_PREFIX + resourceInfo.id);
@@ -1689,7 +1685,7 @@ public class Assembler extends AssemblerTool implements org.apache.openejb.spi.A
}
}
- private void preDestroy(Object object) {
+ private void preDestroy(final Object object) {
final Method preDestroy = findMethodAnnotatedWith(PreDestroy.class, object.getClass());
if (preDestroy != null) {
try {
@@ -1707,7 +1703,7 @@ public class Assembler extends AssemblerTool implements org.apache.openejb.spi.A
resource = LazyResource.class.cast(resource).getObject();
}
- Class<? extends Object> cls = resource.getClass();
+ final Class<? extends Object> cls = resource.getClass();
return findMethodAnnotatedWith(PreDestroy.class, cls) != null;
} catch (Exception e) {
return false;
@@ -2387,7 +2383,7 @@ public class Assembler extends AssemblerTool implements org.apache.openejb.spi.A
}
public void createResource(final ResourceInfo serviceInfo) throws OpenEJBException {
- Object service = "true".equalsIgnoreCase(String.valueOf(serviceInfo.properties.remove("Lazy"))) ?
+ final Object service = "true".equalsIgnoreCase(String.valueOf(serviceInfo.properties.remove("Lazy"))) ?
newLazyResource(serviceInfo) :
doCreateResource(serviceInfo);
http://git-wip-us.apache.org/repos/asf/tomee/blob/ad3b535e/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java
----------------------------------------------------------------------
diff --git a/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java b/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java
index 21c41bc..0a7514b 100644
--- a/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java
+++ b/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java
@@ -25,8 +25,6 @@ import org.apache.openejb.api.RemoteClient;
import org.apache.openejb.classloader.ClassLoaderConfigurer;
import org.apache.openejb.classloader.WebAppEnricher;
import org.apache.openejb.config.event.BeforeDeploymentEvent;
-import org.apache.openejb.config.sys.Resource;
-import org.apache.openejb.config.sys.Resources;
import org.apache.openejb.core.EmptyResourcesClassLoader;
import org.apache.openejb.core.ParentClassLoaderFinder;
import org.apache.openejb.jee.Application;
http://git-wip-us.apache.org/repos/asf/tomee/blob/ad3b535e/container/openejb-core/src/main/java/org/apache/openejb/config/ReadDescriptors.java
----------------------------------------------------------------------
diff --git a/container/openejb-core/src/main/java/org/apache/openejb/config/ReadDescriptors.java b/container/openejb-core/src/main/java/org/apache/openejb/config/ReadDescriptors.java
index 5faeb6f..806793d 100644
--- a/container/openejb-core/src/main/java/org/apache/openejb/config/ReadDescriptors.java
+++ b/container/openejb-core/src/main/java/org/apache/openejb/config/ReadDescriptors.java
@@ -282,6 +282,7 @@ public class ReadDescriptors implements DynamicDeployer {
}
continue;
} catch (Exception e) {
+ // ignore if these classes are found in the classloader
}
// if the resource class cannot be loaded,