[jboss-cvs] JBossAS SVN: r104485 - in projects/jboss-classpool/trunk: jbosscl/src/main/java/org/jboss/classpool/plugins/jbosscl and 3 other directories.
jboss-cvs-commits at lists.jboss.org
jboss-cvs-commits at lists.jboss.org
Wed May 5 12:04:45 EDT 2010
Author: kabir.khan at jboss.com
Date: 2010-05-05 12:04:44 -0400 (Wed, 05 May 2010)
New Revision: 104485
Modified:
projects/jboss-classpool/trunk/classpool/src/main/java/org/jboss/classpool/base/BaseClassPoolDomain.java
projects/jboss-classpool/trunk/jbosscl/src/main/java/org/jboss/classpool/plugins/jbosscl/JBossClDelegatingClassPool.java
projects/jboss-classpool/trunk/jbosscl/src/main/java/org/jboss/classpool/plugins/jbosscl/ParentUnitLoaderFinders.java
projects/jboss-classpool/trunk/jbosscl/src/test/java/org/jboss/test/classpool/jbosscl/support/NoMatchClassFilter.java
projects/jboss-classpool/trunk/jbosscl/src/test/java/org/jboss/test/classpool/jbosscl/test/ConfigClassPoolTestCase.java
projects/jboss-classpool/trunk/jbosscl/src/test/java/org/jboss/test/classpool/jbosscl/test/JBossClClassPoolTestDelegate.java
projects/jboss-classpool/trunk/scopedpool/src/main/java/org/jboss/classpool/scoped/ScopedClassPool.java
projects/jboss-classpool/trunk/scopedpool/src/main/java/org/jboss/classpool/scoped/SoftValueHashMap.java
Log:
Get rid of warnings
Modified: projects/jboss-classpool/trunk/classpool/src/main/java/org/jboss/classpool/base/BaseClassPoolDomain.java
===================================================================
--- projects/jboss-classpool/trunk/classpool/src/main/java/org/jboss/classpool/base/BaseClassPoolDomain.java 2010-05-05 15:48:16 UTC (rev 104484)
+++ projects/jboss-classpool/trunk/classpool/src/main/java/org/jboss/classpool/base/BaseClassPoolDomain.java 2010-05-05 16:04:44 UTC (rev 104485)
@@ -23,7 +23,6 @@
import java.util.List;
import java.util.concurrent.CopyOnWriteArrayList;
-import java.util.concurrent.atomic.AtomicInteger;
import java.util.concurrent.locks.ReadWriteLock;
import java.util.concurrent.locks.ReentrantReadWriteLock;
Modified: projects/jboss-classpool/trunk/jbosscl/src/main/java/org/jboss/classpool/plugins/jbosscl/JBossClDelegatingClassPool.java
===================================================================
--- projects/jboss-classpool/trunk/jbosscl/src/main/java/org/jboss/classpool/plugins/jbosscl/JBossClDelegatingClassPool.java 2010-05-05 15:48:16 UTC (rev 104484)
+++ projects/jboss-classpool/trunk/jbosscl/src/main/java/org/jboss/classpool/plugins/jbosscl/JBossClDelegatingClassPool.java 2010-05-05 16:04:44 UTC (rev 104485)
@@ -41,7 +41,6 @@
class JBossClDelegatingClassPool extends DelegatingClassPool implements ToClassInvokerPoolReference
{
private final WeakReference<Module> module;
- private final RegisterModuleCallback callback;
private final ToClassInvoker toClassInvoker;
@@ -62,7 +61,6 @@
throw new IllegalArgumentException("Domain was not instance of JBossClClassPoolDomain: " + domain.getClass().getName());
}
this.module = new WeakReference<Module>(module);
- this.callback = callback;
toClassInvoker = new ToClassInvoker(module.getDynamicClassRoot());
if (logger.isTraceEnabled()) logger.trace(this + " module");
((JBossClClassPoolDomain)domain).setupPoolsByPackage(this);
Modified: projects/jboss-classpool/trunk/jbosscl/src/main/java/org/jboss/classpool/plugins/jbosscl/ParentUnitLoaderFinders.java
===================================================================
--- projects/jboss-classpool/trunk/jbosscl/src/main/java/org/jboss/classpool/plugins/jbosscl/ParentUnitLoaderFinders.java 2010-05-05 15:48:16 UTC (rev 104484)
+++ projects/jboss-classpool/trunk/jbosscl/src/main/java/org/jboss/classpool/plugins/jbosscl/ParentUnitLoaderFinders.java 2010-05-05 16:04:44 UTC (rev 104485)
@@ -23,7 +23,6 @@
import org.jboss.classloading.spi.dependency.ClassLoading;
import org.jboss.classloading.spi.dependency.Module;
-import org.jboss.deployers.plugins.classloading.AbstractDeploymentClassLoaderPolicyModule;
/**
* Entry point to determining the parent classloader.
@@ -53,8 +52,9 @@
/**
* Calls the registered parent unit loader finders.
- * {@link BootstrapParentUnitFinder} is always available. Once {@link AbstractDeploymentClassLoaderPolicyModule}
- * has been deployed its ParentUnitLoaderFinder is checked if it is of the correct type.
+ * {@link BootstrapParentUnitFinder} is always available. Once AbstractDeploymentClassLoaderPolicyModule
+ * has been deployed its ParentUnitLoaderFinder (AbstractDeploymentClassLoaderPolicyModuleParentUnitLoaderFinder) is checked
+ * if the module is of the correct type.
*/
ClassLoader findParentUnitLoader(Module module)
{
Modified: projects/jboss-classpool/trunk/jbosscl/src/test/java/org/jboss/test/classpool/jbosscl/support/NoMatchClassFilter.java
===================================================================
--- projects/jboss-classpool/trunk/jbosscl/src/test/java/org/jboss/test/classpool/jbosscl/support/NoMatchClassFilter.java 2010-05-05 15:48:16 UTC (rev 104484)
+++ projects/jboss-classpool/trunk/jbosscl/src/test/java/org/jboss/test/classpool/jbosscl/support/NoMatchClassFilter.java 2010-05-05 16:04:44 UTC (rev 104485)
@@ -32,6 +32,8 @@
*/
public class NoMatchClassFilter implements ClassFilter
{
+ private static final long serialVersionUID = 1L;
+
String className;
public boolean filtered = false;
Modified: projects/jboss-classpool/trunk/jbosscl/src/test/java/org/jboss/test/classpool/jbosscl/test/ConfigClassPoolTestCase.java
===================================================================
--- projects/jboss-classpool/trunk/jbosscl/src/test/java/org/jboss/test/classpool/jbosscl/test/ConfigClassPoolTestCase.java 2010-05-05 15:48:16 UTC (rev 104484)
+++ projects/jboss-classpool/trunk/jbosscl/src/test/java/org/jboss/test/classpool/jbosscl/test/ConfigClassPoolTestCase.java 2010-05-05 16:04:44 UTC (rev 104485)
@@ -24,7 +24,6 @@
import static org.jboss.test.classpool.jbosscl.support.SupportArchives.JAR_A_URL;
import static org.jboss.test.classpool.jbosscl.support.SupportArchives.JAR_B_URL;
import static org.jboss.test.classpool.jbosscl.support.SupportClasses.CLASS_A;
-import static org.jboss.test.classpool.jbosscl.support.SupportClasses.CLASS_B;
import java.net.URL;
import java.net.URLClassLoader;
@@ -42,12 +41,10 @@
import org.jboss.beans.metadata.spi.builder.BeanMetaDataBuilder;
import org.jboss.classloader.spi.ClassLoaderSystem;
import org.jboss.classpool.plugins.jbosscl.JBossClClassPoolConfig;
-import org.jboss.classpool.plugins.jbosscl.JBossClDelegatingClassPoolFactory;
import org.jboss.kernel.spi.dependency.KernelControllerContext;
import org.jboss.test.classpool.jbosscl.support.CLDeploymentBuilder;
import org.jboss.test.classpool.support.ClassPoolTestScenario;
import org.jboss.test.classpool.support.TestScenario;
-import org.jboss.util.NotImplementedException;
/**
* Tests the JBossClClassPoolConfig as intended for use in AS
Modified: projects/jboss-classpool/trunk/jbosscl/src/test/java/org/jboss/test/classpool/jbosscl/test/JBossClClassPoolTestDelegate.java
===================================================================
--- projects/jboss-classpool/trunk/jbosscl/src/test/java/org/jboss/test/classpool/jbosscl/test/JBossClClassPoolTestDelegate.java 2010-05-05 15:48:16 UTC (rev 104484)
+++ projects/jboss-classpool/trunk/jbosscl/src/test/java/org/jboss/test/classpool/jbosscl/test/JBossClClassPoolTestDelegate.java 2010-05-05 16:04:44 UTC (rev 104485)
@@ -72,6 +72,8 @@
final String packageName = packageA.substring(0, packageA.lastIndexOf('.'));
ClassFilter classFilter = new ClassFilter()
{
+ private static final long serialVersionUID = 1L;
+
String packagePath = ClassLoaderUtils.packageNameToPath(packageName);
ClassFilter patternFilter = new PatternClassFilter(
new String[] { packageName + "\\..+" },
Modified: projects/jboss-classpool/trunk/scopedpool/src/main/java/org/jboss/classpool/scoped/ScopedClassPool.java
===================================================================
--- projects/jboss-classpool/trunk/scopedpool/src/main/java/org/jboss/classpool/scoped/ScopedClassPool.java 2010-05-05 15:48:16 UTC (rev 104484)
+++ projects/jboss-classpool/trunk/scopedpool/src/main/java/org/jboss/classpool/scoped/ScopedClassPool.java 2010-05-05 16:04:44 UTC (rev 104485)
@@ -97,7 +97,7 @@
}
protected ClassLoader getClassLoader0() {
- return (ClassLoader)classLoader.get();
+ return classLoader.get();
}
/**
@@ -231,7 +231,7 @@
if (cached != null)
return cached;
synchronized (softcache) {
- return (CtClass)softcache.get(classname);
+ return softcache.get(classname);
}
}
Modified: projects/jboss-classpool/trunk/scopedpool/src/main/java/org/jboss/classpool/scoped/SoftValueHashMap.java
===================================================================
--- projects/jboss-classpool/trunk/scopedpool/src/main/java/org/jboss/classpool/scoped/SoftValueHashMap.java 2010-05-05 15:48:16 UTC (rev 104484)
+++ projects/jboss-classpool/trunk/scopedpool/src/main/java/org/jboss/classpool/scoped/SoftValueHashMap.java 2010-05-05 16:04:44 UTC (rev 104485)
@@ -80,7 +80,7 @@
private void processQueue() {
SoftValueRef ref;
while ((ref = (SoftValueRef)queue.poll()) != null) {
- if (ref == (SoftValueRef)hash.get(ref.key)) {
+ if (ref == hash.get(ref.key)) {
// only remove if it is the *exact* same WeakValueRef
//
hash.remove(ref.key);
More information about the jboss-cvs-commits
mailing list