You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jd...@apache.org on 2003/09/01 17:11:57 UTC
cvs commit: incubator-geronimo/modules/common/src/java/org/apache/geronimo/common/jmx MBeanProxyHandler.java
jdillon 2003/09/01 08:11:57
Modified: modules/common/src/java/org/apache/geronimo/common/jmx
MBeanProxyHandler.java
Log:
o use method as key
Revision Changes Path
1.4 +3 -6 incubator-geronimo/modules/common/src/java/org/apache/geronimo/common/jmx/MBeanProxyHandler.java
Index: MBeanProxyHandler.java
===================================================================
RCS file: /home/cvs/incubator-geronimo/modules/common/src/java/org/apache/geronimo/common/jmx/MBeanProxyHandler.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- MBeanProxyHandler.java 30 Aug 2003 20:38:46 -0000 1.3
+++ MBeanProxyHandler.java 1 Sep 2003 15:11:57 -0000 1.4
@@ -297,19 +297,16 @@
protected Task getTask(final Method method, final Object[] args)
throws Exception
{
- // Key off of method name & arg count
- Object key = method.getName() + (args == null ? 0 : args.length);
-
// Check if there is a cached task
Task task = null;
if (taskCache != null) {
- task = (Task)taskCache.get(key);
+ task = (Task)taskCache.get(method);
}
// If not create one and cache it
if (task == null) {
task = createTask(method, args);
- taskCache.put(key, task);
+ taskCache.put(method, task);
}
return task;