You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by mc...@apache.org on 2009/01/08 04:35:52 UTC
svn commit: r732598 -
/geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/MultiParentClassLoader.java
Author: mcconne
Date: Wed Jan 7 19:35:51 2009
New Revision: 732598
URL: http://svn.apache.org/viewvc?rev=732598&view=rev
Log:
GERONIMO-4497 Use less-confusing variable name
Modified:
geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/MultiParentClassLoader.java
Modified: geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/MultiParentClassLoader.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/MultiParentClassLoader.java?rev=732598&r1=732597&r2=732598&view=diff
==============================================================================
--- geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/MultiParentClassLoader.java (original)
+++ geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/MultiParentClassLoader.java Wed Jan 7 19:35:51 2009
@@ -61,7 +61,7 @@
private final String[] hiddenResources;
private final String[] nonOverridableResources;
private boolean destroyed = false;
- private Set<String> knownResources = new HashSet<String>();
+ private Set<String> resourcesNotFound = new HashSet<String>();
// I used this pattern as its temporary and with the static final we get compile time
// optimizations.
@@ -544,7 +544,7 @@
}
public URL getResource(String name) {
- if (isDestroyed() || knownResources.contains(name)) {
+ if (isDestroyed() || resourcesNotFound.contains(name)) {
return null;
}
@@ -587,8 +587,8 @@
//
// Resource not found -- no need to search for it again
//
- if (!knownResources.contains(name)) {
- knownResources.add(name);
+ if (!resourcesNotFound.contains(name)) {
+ resourcesNotFound.add(name);
}
return null;