X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fws%2Fjws1%2FDiscoverer.java;fp=src%2Fjalview%2Fws%2Fjws1%2FDiscoverer.java;h=bee9fadbc634f5631001d72c754d34983799d96e;hb=ec8f3cedf60fb1feed6d34de6b49f6bfa78b9dd8;hp=ea883c86c48cea4512bd850119550a880971fc9e;hpb=056dad85a910551cc95e44d451a61f6b8c4dd35d;p=jalview.git

diff --git a/src/jalview/ws/jws1/Discoverer.java b/src/jalview/ws/jws1/Discoverer.java
index ea883c8..bee9fad 100644
--- a/src/jalview/ws/jws1/Discoverer.java
+++ b/src/jalview/ws/jws1/Discoverer.java
@@ -23,6 +23,7 @@ package jalview.ws.jws1;
 import jalview.gui.JvOptionPane;
 import jalview.util.MessageManager;
 
+import java.net.URL;
 import java.util.Hashtable;
 import java.util.StringTokenizer;
 import java.util.Vector;
@@ -111,21 +112,20 @@ public class Discoverer implements Runnable
 
   static private java.net.URL RootServiceURL = null;
 
-  static public Vector ServiceURLList = null;
+  static public Vector<URL> ServiceURLList = null;
 
   static private boolean reallyDiscoverServices = true;
 
-  public static java.util.Hashtable services = null; // vectors of services
-
+  public static java.util.Hashtable<String, Vector<ServiceHandle>> services = null;
   // stored by
   // abstractServiceType
   // string
 
-  public static java.util.Vector serviceList = null; // flat list of services
+  public static java.util.Vector<ServiceHandle> serviceList = null;
 
-  static private Vector getDiscoveryURLS()
+  static private Vector<URL> getDiscoveryURLS()
   {
-    Vector urls = new Vector();
+    Vector<URL> urls = new Vector<>();
     String RootServiceURLs = jalview.bin.Cache.getDefault("DISCOVERY_URLS",
             "http://www.compbio.dundee.ac.uk/JalviewWS/services/ServiceRegistry");
 
@@ -189,7 +189,7 @@ public class Discoverer implements Runnable
       else
       {
         jalview.bin.Cache.log.debug("Setting default services");
-        services = new Hashtable();
+        services = new Hashtable<>();
         // Muscle, Clustal and JPred.
         ServiceHandle[] defServices = { new ServiceHandle("MsaWS",
                 "Edgar, Robert C. (2004), MUSCLE: multiple sequence alignment "
@@ -217,8 +217,8 @@ public class Discoverer implements Runnable
                             + "\ndoi://10.1093/nar/gkv332",
                     "http://www.compbio.dundee.ac.uk/JalviewWS/services/jpred",
                     "JPred Secondary Structure Prediction") };
-        services = new Hashtable();
-        serviceList = new Vector();
+        services = new Hashtable<>();
+        serviceList = new Vector<>();
         buildServiceLists(defServices, serviceList, services);
       }
 
@@ -284,8 +284,9 @@ public class Discoverer implements Runnable
    *          Hashtable
    * @return boolean
    */
-  static private boolean buildServiceLists(ServiceHandle[] sh, Vector cat,
-          Hashtable sscat)
+  static private boolean buildServiceLists(ServiceHandle[] sh,
+          Vector<ServiceHandle> cat,
+          Hashtable<String, Vector<ServiceHandle>> sscat)
   {
     boolean seenNewDiscovery = false;
     for (int i = 0, j = sh.length; i < j; i++)
@@ -297,11 +298,11 @@ public class Discoverer implements Runnable
                 + sh[i].getEndpointURL() + "\n");
         if (!sscat.containsKey(sh[i].getAbstractName()))
         {
-          sscat.put(sh[i].getAbstractName(), cat = new Vector());
+          sscat.put(sh[i].getAbstractName(), cat = new Vector<>());
         }
         else
         {
-          cat = (Vector) sscat.get(sh[i].getAbstractName());
+          cat = sscat.get(sh[i].getAbstractName());
         }
         cat.add(sh[i]);
         if (sh[i].getAbstractName().equals("Registry"))
@@ -334,8 +335,8 @@ public class Discoverer implements Runnable
 
   public void discoverServices()
   {
-    Hashtable sscat = new Hashtable();
-    Vector cat = new Vector();
+    Hashtable<String, Vector<ServiceHandle>> sscat = new Hashtable<>();
+    Vector<ServiceHandle> cat = new Vector<>();
     ServiceHandle sh[] = null;
     int s_url = 0;
     if (ServiceURLList == null)
@@ -347,7 +348,7 @@ public class Discoverer implements Runnable
     while (s_url < ServiceURLList.size())
     {
       if ((sh = getServices(
-              (java.net.URL) ServiceURLList.get(s_url))) != null)
+              ServiceURLList.get(s_url))) != null)
       {
 
         buildServiceLists(sh, cat, sscat);
@@ -365,7 +366,7 @@ public class Discoverer implements Runnable
     // so no need to access original discovery thread.
     // Curent decision is to change properties then notify listeners with old
     // and new values.
-    Hashtable oldServices = services;
+    Hashtable<String, Vector<ServiceHandle>> oldServices = services;
     // Vector oldServicelist = serviceList;
     services = sscat;
     serviceList = cat;
@@ -384,7 +385,7 @@ public class Discoverer implements Runnable
       @Override
       public void run()
       {
-        discoverer.doDiscovery();
+        Discoverer.doDiscovery();
         discoverer.discoverServices();
       }
     };
@@ -394,19 +395,19 @@ public class Discoverer implements Runnable
   /**
    * binding service abstract name to handler class
    */
-  private static Hashtable serviceClientBindings;
+  private static Hashtable<String, WS1Client> serviceClientBindings;
 
   public static WS1Client getServiceClient(ServiceHandle sh)
   {
     if (serviceClientBindings == null)
     {
       // get a list from Config or create below
-      serviceClientBindings = new Hashtable();
+      serviceClientBindings = new Hashtable<>();
       serviceClientBindings.put("MsaWS", new MsaWSClient());
       serviceClientBindings.put("SecStrPred", new JPredClient());
       serviceClientBindings.put("SeqSearch", new SeqSearchWSClient());
     }
-    WS1Client instance = (WS1Client) serviceClientBindings
+    WS1Client instance = serviceClientBindings
             .get(sh.getAbstractName());
     if (instance == null)
     {