This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/master by this push:
new 8b88823 Remove unused code
8b88823 is described below
commit 8b8882349f0634467527404d0e5b74043d020708
Author: Mark Thomas <[email protected]>
AuthorDate: Tue Apr 23 11:09:08 2019 +0100
Remove unused code
---
java/org/apache/tomcat/util/modeler/Registry.java | 34 ++---------------------
1 file changed, 2 insertions(+), 32 deletions(-)
diff --git a/java/org/apache/tomcat/util/modeler/Registry.java
b/java/org/apache/tomcat/util/modeler/Registry.java
index c116b82..bea9c51 100644
--- a/java/org/apache/tomcat/util/modeler/Registry.java
+++ b/java/org/apache/tomcat/util/modeler/Registry.java
@@ -76,11 +76,6 @@ public class Registry implements RegistryMBean,
MBeanRegistration {
// Support for the factory methods
/**
- * Will be used to isolate different apps and enhance security.
- */
- private static final HashMap<Object, Registry> perLoaderRegistries = null;
-
- /**
* The registry instance created by our factory method the first time it is
* called.
*/
@@ -131,38 +126,13 @@ public class Registry implements RegistryMBean,
MBeanRegistration {
* Factory method to create (if necessary) and return our
* <code>Registry</code> instance.
*
- * The current version uses a static - future versions could use the thread
- * class loader.
- *
- * @param key Support for application isolation. If null, the context class
- * loader will be used ( if setUseContextClassLoader is called )
- * or the default registry is returned.
+ * @param key Unused
* @param guard Prevent access to the registry by untrusted components
+ *
* @return the registry
* @since 1.1
*/
public static synchronized Registry getRegistry(Object key, Object guard) {
- Registry localRegistry;
- if (perLoaderRegistries != null) {
- if (key == null)
- key = Thread.currentThread().getContextClassLoader();
- if (key != null) {
- localRegistry = perLoaderRegistries.get(key);
- if (localRegistry == null) {
- localRegistry = new Registry();
- // localRegistry.key=key;
- localRegistry.guard = guard;
- perLoaderRegistries.put(key, localRegistry);
- return localRegistry;
- }
- if (localRegistry.guard != null && localRegistry.guard !=
guard) {
- return null; // XXX Should I throw a permission ex ?
- }
- return localRegistry;
- }
- }
-
- // static
if (registry == null) {
registry = new Registry();
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]