You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2022/09/15 16:25:49 UTC
[tomcat] branch 10.0.x updated: Refactor to avoid use of Hashtable where possible. No functional change.
This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch 10.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/10.0.x by this push:
new dd1019ec90 Refactor to avoid use of Hashtable where possible. No functional change.
dd1019ec90 is described below
commit dd1019ec9039dc0212468e2ea218406de32563d3
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Thu Sep 15 17:25:35 2022 +0100
Refactor to avoid use of Hashtable where possible. No functional change.
---
java/org/apache/tomcat/util/IntrospectionUtils.java | 4 +++-
test/org/apache/catalina/tribes/demos/IntrospectionUtils.java | 5 +++--
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/java/org/apache/tomcat/util/IntrospectionUtils.java b/java/org/apache/tomcat/util/IntrospectionUtils.java
index ff13d115a1..c5da1b5f45 100644
--- a/java/org/apache/tomcat/util/IntrospectionUtils.java
+++ b/java/org/apache/tomcat/util/IntrospectionUtils.java
@@ -21,6 +21,8 @@ import java.lang.reflect.Method;
import java.net.InetAddress;
import java.net.UnknownHostException;
import java.util.Hashtable;
+import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
import org.apache.juli.logging.Log;
import org.apache.juli.logging.LogFactory;
@@ -407,7 +409,7 @@ public final class IntrospectionUtils {
objectMethods.clear();
}
- private static final Hashtable<Class<?>,Method[]> objectMethods = new Hashtable<>();
+ private static final Map<Class<?>,Method[]> objectMethods = new ConcurrentHashMap<>();
public static Method[] findMethods(Class<?> c) {
Method methods[] = objectMethods.get(c);
diff --git a/test/org/apache/catalina/tribes/demos/IntrospectionUtils.java b/test/org/apache/catalina/tribes/demos/IntrospectionUtils.java
index 816a1ae5c9..885d56701c 100644
--- a/test/org/apache/catalina/tribes/demos/IntrospectionUtils.java
+++ b/test/org/apache/catalina/tribes/demos/IntrospectionUtils.java
@@ -20,7 +20,8 @@ import java.lang.reflect.InvocationTargetException;
import java.lang.reflect.Method;
import java.net.InetAddress;
import java.net.UnknownHostException;
-import java.util.Hashtable;
+import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
import org.apache.juli.logging.Log;
import org.apache.juli.logging.LogFactory;
@@ -209,7 +210,7 @@ public final class IntrospectionUtils {
objectMethods.clear();
}
- static Hashtable<Class<?>,Method[]> objectMethods = new Hashtable<>();
+ static Map<Class<?>,Method[]> objectMethods = new ConcurrentHashMap<>();
public static Method[] findMethods(Class<?> c) {
Method methods[] = objectMethods.get(c);
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org