You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicemix.apache.org by gn...@apache.org on 2007/03/29 19:25:33 UTC

svn commit: r523773 - in /incubator/servicemix/trunk/core/servicemix-services/src/main/java/org/apache/servicemix: executors/impl/ExecutorFactoryImpl.java locks/impl/SimpleLockManager.java

Author: gnodet
Date: Thu Mar 29 10:25:32 2007
New Revision: 523773

URL: http://svn.apache.org/viewvc?view=rev&rev=523773
Log:
SM-903: use generics

Modified:
    incubator/servicemix/trunk/core/servicemix-services/src/main/java/org/apache/servicemix/executors/impl/ExecutorFactoryImpl.java
    incubator/servicemix/trunk/core/servicemix-services/src/main/java/org/apache/servicemix/locks/impl/SimpleLockManager.java

Modified: incubator/servicemix/trunk/core/servicemix-services/src/main/java/org/apache/servicemix/executors/impl/ExecutorFactoryImpl.java
URL: http://svn.apache.org/viewvc/incubator/servicemix/trunk/core/servicemix-services/src/main/java/org/apache/servicemix/executors/impl/ExecutorFactoryImpl.java?view=diff&rev=523773&r1=523772&r2=523773
==============================================================================
--- incubator/servicemix/trunk/core/servicemix-services/src/main/java/org/apache/servicemix/executors/impl/ExecutorFactoryImpl.java (original)
+++ incubator/servicemix/trunk/core/servicemix-services/src/main/java/org/apache/servicemix/executors/impl/ExecutorFactoryImpl.java Thu Mar 29 10:25:32 2007
@@ -51,7 +51,7 @@
 public class ExecutorFactoryImpl implements ExecutorFactory {
 
     private ExecutorConfig defaultConfig = new ExecutorConfig();
-    private Map configs = new HashMap();
+    private Map<String,ExecutorConfig> configs = new HashMap<String, ExecutorConfig>();
     
     public Executor createExecutor(String id) {
         ExecutorConfig config = getConfig(id);
@@ -62,10 +62,10 @@
     protected ExecutorConfig getConfig(String id) {
         ExecutorConfig config = null;
         if (configs != null) {
-            config = (ExecutorConfig) configs.get(id);
+            config = configs.get(id);
             while (config == null && id.indexOf('.') > 0) {
                 id = id.substring(0, id.lastIndexOf('.'));
-                config = (ExecutorConfig) configs.get(id);
+                config = configs.get(id);
             }
         }
         if (config == null) {
@@ -78,13 +78,13 @@
         if (config.getQueueSize() != 0 && config.getCorePoolSize() == 0) {
             throw new IllegalArgumentException("CorePoolSize must be > 0 when using a capacity queue");
         }
-        BlockingQueue queue;
+        BlockingQueue<Runnable> queue;
         if (config.getQueueSize() == 0) {
-            queue = new SynchronousQueue();
+            queue = new SynchronousQueue<Runnable>();
         } else if (config.getQueueSize() < 0 || config.getQueueSize() == Integer.MAX_VALUE) {
-            queue = new LinkedBlockingQueue();
+            queue = new LinkedBlockingQueue<Runnable>();
         } else {
-            queue = new ArrayBlockingQueue(config.getQueueSize());
+            queue = new ArrayBlockingQueue<Runnable>(config.getQueueSize());
         }
         ThreadFactory factory = new DefaultThreadFactory(id,
                                                          config.isThreadDaemon(), 
@@ -144,14 +144,14 @@
     /**
      * @return the configs
      */
-    public Map getConfigs() {
+    public Map<String, ExecutorConfig> getConfigs() {
         return configs;
     }
 
     /**
      * @param configs the configs to set
      */
-    public void setConfigs(Map configs) {
+    public void setConfigs(Map<String, ExecutorConfig> configs) {
         this.configs = configs;
     }
 

Modified: incubator/servicemix/trunk/core/servicemix-services/src/main/java/org/apache/servicemix/locks/impl/SimpleLockManager.java
URL: http://svn.apache.org/viewvc/incubator/servicemix/trunk/core/servicemix-services/src/main/java/org/apache/servicemix/locks/impl/SimpleLockManager.java?view=diff&rev=523773&r1=523772&r2=523773
==============================================================================
--- incubator/servicemix/trunk/core/servicemix-services/src/main/java/org/apache/servicemix/locks/impl/SimpleLockManager.java (original)
+++ incubator/servicemix/trunk/core/servicemix-services/src/main/java/org/apache/servicemix/locks/impl/SimpleLockManager.java Thu Mar 29 10:25:32 2007
@@ -25,13 +25,13 @@
 
 public class SimpleLockManager implements LockManager {
 
-    private ConcurrentMap locks = new ConcurrentHashMap();
+    private ConcurrentMap<String, Lock> locks = new ConcurrentHashMap<String, Lock>();
     
     public Lock getLock(String id) {
-        Lock lock = (Lock) locks.get(id);
+        Lock lock = locks.get(id);
         if (lock == null) {
             lock = new ReentrantLock();
-            Lock oldLock = (Lock) locks.putIfAbsent(id, lock);
+            Lock oldLock = locks.putIfAbsent(id, lock);
             if (oldLock != null) {
                 lock = oldLock;
             }