You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by mc...@apache.org on 2008/12/09 13:47:01 UTC

svn commit: r724672 - /tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/invocation/CachedProxy.java

Author: mcombellack
Date: Tue Dec  9 04:47:01 2008
New Revision: 724672

URL: http://svn.apache.org/viewvc?rev=724672&view=rev
Log:
Fixed generics raw type compiler warnings

Modified:
    tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/invocation/CachedProxy.java

Modified: tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/invocation/CachedProxy.java
URL: http://svn.apache.org/viewvc/tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/invocation/CachedProxy.java?rev=724672&r1=724671&r2=724672&view=diff
==============================================================================
--- tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/invocation/CachedProxy.java (original)
+++ tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/invocation/CachedProxy.java Tue Dec  9 04:47:01 2008
@@ -34,20 +34,20 @@
     // This is a cache containing the proxy class constructor for each business interface.
     // This improves performance compared to calling Proxy.newProxyInstance()
     // every time that a proxy is needed.
-    private static WeakHashMap<Class<?>, Object> cache = new WeakHashMap<Class<?>, Object>();
+    private final static WeakHashMap<Class<?>, Constructor<?>> cache = new WeakHashMap<Class<?>, Constructor<?>>();
 
-    public static Object newProxyInstance(ClassLoader classloader, Class aclass[], InvocationHandler invocationhandler)
+    public static Object newProxyInstance(ClassLoader classloader, Class<?> aclass[], InvocationHandler invocationhandler)
         throws IllegalArgumentException {
         try {
             if (invocationhandler == null)
                 throw new NullPointerException();
             // Lookup cached constructor.  aclass[0] is the reference's business interface.
-            Constructor proxyCTOR;
+            Constructor<?> proxyCTOR;
             synchronized (cache) {
-                proxyCTOR = (Constructor)cache.get(aclass[0]);
+                proxyCTOR = cache.get(aclass[0]);
             }
             if (proxyCTOR == null) {
-                Class proxyClass = getProxyClass(classloader, aclass);
+                Class<?> proxyClass = getProxyClass(classloader, aclass);
                 proxyCTOR = proxyClass.getConstructor(constructorParams);
                 synchronized (cache) {
                     cache.put(aclass[0], proxyCTOR);