Revision: 15160
http://gate.svn.sourceforge.net/gate/?rev=15160&view=rev
Author: markagreenwood
Date: 2012-01-21 12:22:41 +0000 (Sat, 21 Jan 2012)
Log Message:
-----------
some generics stuff and make it clear that we ignore runtime exceptions as well
as checked exceptions when trying to generate the name of a resource
Modified Paths:
--------------
gate/trunk/src/gate/Factory.java
Modified: gate/trunk/src/gate/Factory.java
===================================================================
--- gate/trunk/src/gate/Factory.java 2012-01-21 12:10:12 UTC (rev 15159)
+++ gate/trunk/src/gate/Factory.java 2012-01-21 12:22:41 UTC (rev 15160)
@@ -46,6 +46,7 @@
import java.net.URI;
import java.net.URL;
import java.util.Collection;
+import java.util.EventListener;
import java.util.HashMap;
import java.util.IdentityHashMap;
import java.util.Map;
@@ -60,9 +61,6 @@
/** An object to source events from. */
private static CreoleProxy creoleProxy;
- /** An object to source events from. */
- private static HashMap accessControllerPool;
-
/** Create an instance of a resource using default parameter values.
* @see #createResource(String,FeatureMap)
*/
@@ -342,10 +340,12 @@
}
}
}
- }catch(Exception t){
+ } catch (RuntimeException t) {
+ //even runtime exceptions are safe to ignore at this point
+ } catch(Exception t) {
//there were problems while trying to guess a name
//we can safely ignore them
- }finally{
+ } finally{
//make sure there is a name provided, whatever happened
if(resourceName == null || resourceName.trim().length() == 0){
resourceName = resData.getName();
@@ -356,7 +356,7 @@
} // else if(res.getName() == null)
// if res.getName() != null, leave it as it is
- Map listeners = new HashMap(gate.Gate.getListeners());
+ Map<String, EventListener> listeners = new HashMap<String,
EventListener>(gate.Gate.getListeners());
// set the listeners if any
if(listeners != null && !listeners.isEmpty()) {
try {
@@ -747,19 +747,20 @@
}
}
- static Class japeParserClass = ParseCpsl.class;
- public static Class getJapeParserClass() {
+ static Class<ParseCpsl> japeParserClass = ParseCpsl.class;
+ public static Class<ParseCpsl> getJapeParserClass() {
return japeParserClass;
}
+ @SuppressWarnings("unchecked")
public static void setJapeParserClass(Class newClass) {
if (! ParseCpsl.class.isAssignableFrom(newClass))
throw new IllegalArgumentException("Parser class must inherit from "
+ ParseCpsl.class);
japeParserClass = newClass;
}
- public static ParseCpsl newJapeParser(java.io.Reader stream, HashMap
existingMacros) {
+ public static ParseCpsl newJapeParser(java.io.Reader stream, Map
existingMacros) {
try {
- Constructor c = japeParserClass.getConstructor
+ Constructor<ParseCpsl> c = japeParserClass.getConstructor
(new Class[] {java.io.Reader.class, existingMacros.getClass()});
return (ParseCpsl) c.newInstance(new Object[] {stream,
existingMacros});
} catch (NoSuchMethodException e) { // Shouldn't happen
@@ -861,7 +862,6 @@
/** Static initialiser to set up the CreoleProxy event source object */
static {
creoleProxy = new CreoleProxy();
- accessControllerPool = new HashMap();
} // static initialiser
} // abstract Factory
@@ -876,15 +876,17 @@
public synchronized void removeCreoleListener(CreoleListener l) {
if (creoleListeners != null && creoleListeners.contains(l)) {
- Vector v = (Vector) creoleListeners.clone();
+ @SuppressWarnings("unchecked")
+ Vector<CreoleListener> v = (Vector<CreoleListener>)
creoleListeners.clone();
v.removeElement(l);
creoleListeners = v;
}// if
}// removeCreoleListener(CreoleListener l)
public synchronized void addCreoleListener(CreoleListener l) {
- Vector v =
- creoleListeners == null ? new Vector(2) : (Vector)
creoleListeners.clone();
+ @SuppressWarnings("unchecked")
+ Vector<CreoleListener> v =
+ creoleListeners == null ? new Vector<CreoleListener>(2) :
(Vector<CreoleListener>) creoleListeners.clone();
if (!v.contains(l)) {
v.addElement(l);
creoleListeners = v;
@@ -893,53 +895,48 @@
protected void fireResourceLoaded(CreoleEvent e) {
if (creoleListeners != null) {
- Vector listeners = creoleListeners;
- int count = listeners.size();
+ int count = creoleListeners.size();
for (int i = 0; i < count; i++) {
- ((CreoleListener) listeners.elementAt(i)).resourceLoaded(e);
+ creoleListeners.elementAt(i).resourceLoaded(e);
}// for
}// if
}// fireResourceLoaded(CreoleEvent e)
protected void fireResourceUnloaded(CreoleEvent e) {
if (creoleListeners != null) {
- Vector listeners = creoleListeners;
- int count = listeners.size();
+ int count = creoleListeners.size();
for (int i = 0; i < count; i++) {
- ((CreoleListener) listeners.elementAt(i)).resourceUnloaded(e);
+ creoleListeners.elementAt(i).resourceUnloaded(e);
}// for
}// if
}// fireResourceUnloaded(CreoleEvent e)
protected void fireDatastoreOpened(CreoleEvent e) {
if (creoleListeners != null) {
- Vector listeners = creoleListeners;
- int count = listeners.size();
+ int count = creoleListeners.size();
for (int i = 0; i < count; i++) {
- ((CreoleListener) listeners.elementAt(i)).datastoreOpened(e);
+ creoleListeners.elementAt(i).datastoreOpened(e);
}// for
}// if
}// fireDatastoreOpened(CreoleEvent e)
protected void fireDatastoreCreated(CreoleEvent e) {
if (creoleListeners != null) {
- Vector listeners = creoleListeners;
- int count = listeners.size();
+ int count = creoleListeners.size();
for (int i = 0; i < count; i++) {
- ((CreoleListener) listeners.elementAt(i)).datastoreCreated(e);
+ creoleListeners.elementAt(i).datastoreCreated(e);
}// for
}// if
}// fireDatastoreCreated(CreoleEvent e)
protected void fireDatastoreClosed(CreoleEvent e) {
if (creoleListeners != null) {
- Vector listeners = creoleListeners;
- int count = listeners.size();
+ int count = creoleListeners.size();
for (int i = 0; i < count; i++) {
- ((CreoleListener) listeners.elementAt(i)).datastoreClosed(e);
+ creoleListeners.elementAt(i).datastoreClosed(e);
}// for
}// if
}// fireDatastoreClosed(CreoleEvent e)
- private transient Vector creoleListeners;
+ private transient Vector<CreoleListener> creoleListeners;
}//class CreoleProxy
This was sent by the SourceForge.net collaborative development platform, the
world's largest Open Source development site.
------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2
_______________________________________________
GATE-cvs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/gate-cvs