Polish generics

ajbrowser/src/org/aspectj/tools/ajbrowser/core/BrowserCompilerConfiguration.java

 		return classpath.toString();
 	}
 
-	public Map getJavaOptionsMap() {
+	public Map<String,String> getJavaOptionsMap() {
 		return BrowserManager.getDefault().getJavaBuildOptions().getJavaBuildOptionsMap();
 	}
 

ajde.core/src/org/aspectj/ajde/core/ICompilerConfiguration.java

 	 * @see org.aspectj.ajde.core.JavaOptions#getDefaultJavaOptions or
 	 *      org.aspectj.org.eclipse.jdt.core.IJavaProject#getOptions(boolean)
 	 */
-	public Map /* String --> String */getJavaOptionsMap();
+	public Map<String,String> getJavaOptionsMap();
 
 	/**
 	 * The non-standard options, typically prefaced with -X when used with a command line compiler. The default is no non-standard
 	 * 
 	 * @return a subset of those files that would be returned on getProjectSourceFiles() that have actually *changed*
 	 */
-	public List /* File */getProjectSourceFilesChanged();
+	public List<File> getProjectSourceFilesChanged();
 
 	/**
 	 * @return the classpath to use
 	 * 
 	 * @return map from unique resource name to absolute path to source resource (String to File)
 	 */
-	public Map /* String --> java.io.File */getSourcePathResources();
+	public Map<String,File> getSourcePathResources();
 
 	/**
 	 * Returns a set of bit flags indicating what has changed in the configuration since it was previously read. This allows the
 	 * @return a list of modified elements that should be checked (can be empty) or null if unknown (and in which case every
 	 *         classpath element will be checked)
 	 */
-	public List getClasspathElementsWithModifiedContents();
+	public List<String> getClasspathElementsWithModifiedContents();
 
 	//
 	// /**

ajde.core/src/org/aspectj/ajde/core/JavaOptions.java

 	public static final String GENERATE = CompilerOptions.GENERATE;
 	public static final String DO_NOT_GENERATE = CompilerOptions.DO_NOT_GENERATE;
 	
-	private static Map defaultOptionsMap;
+	private static Map<String,String> defaultOptionsMap;
 	
 	/**
 	 * @return the java options map with the default settings
 	 */
-	public static Map getDefaultJavaOptions() {
+	public static Map<String,String> getDefaultJavaOptions() {
 		if (defaultOptionsMap != null) return defaultOptionsMap;
 		
-		defaultOptionsMap = new HashMap();
+		defaultOptionsMap = new HashMap<String,String>();
 		defaultOptionsMap.put(COMPLIANCE_LEVEL, VERSION_14);
 		defaultOptionsMap.put(SOURCE_COMPATIBILITY_LEVEL, VERSION_13);
 		defaultOptionsMap.put(PRESERVE_ALL_LOCALS, OPTIMIZE);

ajde.core/src/org/aspectj/ajde/core/internal/AjdeCoreBuildManager.java

 		return formattedOptions.toString();
 	}
 
-	private String formatMap(Map options) {
+	private String formatMap(Map<String,? extends Object> options) {
 		if (options == null) {
 			return "<default>";
 		}
 		mergeInto(config.getAspectpath(), compilerConfig.getAspectPath());
 
 		// Process the JAVA OPTIONS MAP
-		Map jom = compilerConfig.getJavaOptionsMap();
+		Map<String,String> jom = compilerConfig.getJavaOptionsMap();
 		if (jom != null) {
 			String version = (String) jom.get(CompilerOptions.OPTION_Compliance);
 			if (version != null && !version.equals(CompilerOptions.VERSION_1_4)) {

ajde.core/src/org/aspectj/ajde/core/internal/OutputLocationAdapter.java

 		return this.locationManager.getOutputLocationForResource(resource);
 	}
 
-	public List getAllOutputLocations() {
+	public List<File> getAllOutputLocations() {
 		return this.locationManager.getAllOutputLocations();
 	}
 
 	 * 
 	 * @return a map from inpath entries (jars/dirs) to handle components.
 	 */
-	public Map getInpathMap() {
+	public Map<File,String> getInpathMap() {
 		return this.locationManager.getInpathMap();
 	}
 

ajde.core/testsrc/org/aspectj/ajde/core/AjdeCoreTestCase.java

 import java.io.File;
 import java.io.IOException;
 import java.util.ArrayList;
-import java.util.Iterator;
 import java.util.List;
 
 import junit.framework.AssertionFailedError;
 import junit.framework.TestCase;
 
+import org.aspectj.ajde.core.TestMessageHandler.TestMessage;
 import org.aspectj.tools.ajc.Ajc;
 
 /**
 	}
 
 	public boolean checkFor(String what) {
-		List ll = ((TestMessageHandler) compiler.getMessageHandler()).getMessages();
-		for (Iterator iter = ll.iterator(); iter.hasNext();) {
-			Object element = iter.next();
+		List<TestMessage> ll = ((TestMessageHandler) compiler.getMessageHandler()).getMessages();
+		for (TestMessage element: ll) {
 			if (element.toString().indexOf(what) != -1)
 				return true;
 		}
 	}
 
 	public void dumpTaskData() {
-		List ll = ((TestMessageHandler) compiler.getMessageHandler()).getMessages();
-		for (Iterator iter = ll.iterator(); iter.hasNext();) {
-			Object element = iter.next();
+		List<TestMessage> ll = ((TestMessageHandler) compiler.getMessageHandler()).getMessages();
+		for (TestMessage element: ll) {
 			System.out.println("RecordedMessage>" + element);
 		}
 	}

ajde.core/testsrc/org/aspectj/ajde/core/TestBuildProgressMonitor.java

 package org.aspectj.ajde.core;
 
 import java.util.ArrayList;
-import java.util.Iterator;
 import java.util.List;
 
 /**
     
 	private String programmableString;
 	private int count;
-	private List messagesReceived = new ArrayList();
+	private List<String> messagesReceived = new ArrayList<String>();
 	private int currentVal;
 	private boolean isCancelRequested = false;
 		
 	}
 	
 	public boolean containsMessage(String prefix,String distinguishingMarks) {
-		for (Iterator iter = messagesReceived.iterator(); iter.hasNext();) {
-			String element = (String) iter.next();
+		for (String element: messagesReceived) {
 			if (element.startsWith(prefix) &&
 			    element.indexOf(distinguishingMarks)!=-1) return true;
 		}
 	
 	public void dumpMessages() {
 		System.out.println("ProgressMonitorMessages");
-		for (Iterator iter = messagesReceived.iterator(); iter.hasNext();) {
-			String element = (String) iter.next();
+		for (String element: messagesReceived) {
 			System.out.println(element);
 		}
 	}
-
 	
 }

ajde.core/testsrc/org/aspectj/ajde/core/TestCompilerConfiguration.java

 		return projectSourceFiles;
 	}
 
-	public List getProjectSourceFilesChanged() {
+	public List<File> getProjectSourceFilesChanged() {
 		return null;
 	}
 
 		this.outjar = outjar;
 	}
 
-	public void setJavaOptions(Map javaOptions) {
+	public void setJavaOptions(Map<String,String> javaOptions) {
 		this.javaOptions = javaOptions;
 	}
 
 		return ICompilerConfiguration.EVERYTHING;
 	}
 
-	public List getClasspathElementsWithModifiedContents() {
+	public List<String> getClasspathElementsWithModifiedContents() {
 		return null;
 	}
 

ajde.core/testsrc/org/aspectj/ajde/core/TestMessageHandler.java

 public class TestMessageHandler implements IBuildMessageHandler {
 
 	private List<Kind> ignoring;
-	private List messages;
-	private List errors;
+	private List<TestMessage> messages;
+	private List<TestMessage> errors;
 
 	public TestMessageHandler() {
 		ignoring = new ArrayList<Kind>();
-		messages = new ArrayList();
-		errors = new ArrayList();
+		messages = new ArrayList<TestMessage>();
+		errors = new ArrayList<TestMessage>();
 		ignore(IMessage.INFO);
 		ignore(IMessage.WEAVEINFO);
 	}
 		}
 	}
 
-	public List getMessages() {
+	public List<TestMessage> getMessages() {
 		return messages;
 	}
 
-	public List getErrors() {
+	public List<TestMessage> getErrors() {
 		return errors;
 	}
 

ajde.core/testsrc/org/aspectj/ajde/core/TestOutputLocationManager.java

 		resourceOutputLoc = f;
 	}
 
-	public List getAllOutputLocations() {
+	public List<File> getAllOutputLocations() {
 		if (allOutputLocations == null) {
-			allOutputLocations = new ArrayList();
+			allOutputLocations = new ArrayList<File>();
 			initLocations();
 			allOutputLocations.add(classOutputLoc);
 			if (!classOutputLoc.equals(resourceOutputLoc)) {

ajde.core/testsrc/org/aspectj/ajde/core/tests/AjConfigTests.java

 	}
 
 	public void testJavaOptionsMap() {
-		Map options = JavaOptions.getDefaultJavaOptions();
+		Map<String,String> options = JavaOptions.getDefaultJavaOptions();
 		options.put(JavaOptions.WARN_DEPRECATION, JavaOptions.WARNING);
 		compilerConfig.setJavaOptions(options);
 		Map found = genAjBuildConfig().getOptions().getMap();

ajde.core/testsrc/org/aspectj/ajde/core/tests/CompilerMessagesTests.java

 import org.aspectj.ajde.core.AjdeCoreTestCase;
 import org.aspectj.ajde.core.TestCompilerConfiguration;
 import org.aspectj.ajde.core.TestMessageHandler;
+import org.aspectj.ajde.core.TestMessageHandler.TestMessage;
 import org.aspectj.bridge.IMessage;
 
 public class CompilerMessagesTests extends AjdeCoreTestCase {
 		// bug 33474
 		// The build has happened, what messages did the compiler give, and do they
 		// contain the information we expect?
-		List msgs = handler.getMessages();
+		List<TestMessage> msgs = handler.getMessages();
         if (2 != msgs.size()) {
             assertTrue("not two messages: " + msgs, false);
         }
 	}
 	
 	public void testDeclareMessageContents() {
-		List msgs = handler.getMessages();
+		List<TestMessage> msgs = handler.getMessages();
 		IMessage msg = ((TestMessageHandler.TestMessage)msgs.get(1)).getContainedMessage();
 		assertEquals( "Please don't call setters" , msg.getMessage());
 		assertEquals("field-set(int apackage.SomeClass.x)", msg.getDetails());

ajde.core/testsrc/org/aspectj/ajde/core/tests/DuplicateManifestTests.java

 	}
 
 	public void testWeave() {
-		Set injars = new HashSet();
+		Set<File> injars = new HashSet<File>();
 		injars.add(openFile(injarName));
 		compilerConfig.setInpath(injars);
-		Set aspectpath = new HashSet();
+		Set<File> aspectpath = new HashSet<File>();
 		aspectpath.add(openFile(aspectjarName));
 		compilerConfig.setAspectPath(aspectpath);
 		File outjar = openFile(outjarName);

ajde.core/testsrc/org/aspectj/ajde/core/tests/InpathTests.java

 	 * 
 	 */
 	public void testInpathToBin() {
-		Set inpath = new HashSet();
+		Set<File> inpath = new HashSet<File>();
 		File indir1 = openFile(indir1Name);
 		inpath.add(indir1);
 		compilerConfig.setInpath(inpath);
 		doBuild(true);
 		assertTrue("Expected no compiler errors or warnings but found " + handler.getMessages(), handler.getMessages().isEmpty());
 
-		Set expectedBindirContents = new HashSet();
+		Set<String> expectedBindirContents = new HashSet<String>();
 		// From indir1
 		// If we don't copy resources, these next three files won't make it
 		// expectedBindirContents.add("META-INF/MANIFEST.MF");
 	/*
 	 * Ensure -outjar contains all non-Java resouces from injars
 	 */
-	public void compareJars(File dirFile, String sourceDir, File outjarFile, Set expectedOutputJarContents) {
+	public void compareJars(File dirFile, String sourceDir, File outjarFile, Set<String> expectedOutputJarContents) {
 
 		try {
 			assertTrue(

ajde.core/testsrc/org/aspectj/ajde/core/tests/OutxmlTests.java

 import org.aspectj.ajde.core.AjdeCoreTestCase;
 import org.aspectj.ajde.core.TestCompilerConfiguration;
 import org.aspectj.ajde.core.TestMessageHandler;
+import org.aspectj.ajde.core.TestMessageHandler.TestMessage;
 import org.aspectj.bridge.Constants;
 
 public class OutxmlTests extends AjdeCoreTestCase {
 	 */
 	public void testOutxmlToOutjarWithAop_xml () {
 		File f = new File( getAbsoluteProjectDir() + File.separator + "src-resources" + File.separator + "testjar.jar");
-		Set roots = new HashSet();
+		Set<File> roots = new HashSet<File>();
 		roots.add(f);
 		compilerConfig.setInpath(roots);
 		
 		assertFalse("Expected compiler errors or warnings but didn't find any "
 				+ handler.getMessages(), handler.getMessages().isEmpty());
 		
-		List msgs = handler.getMessages();
+		List<TestMessage> msgs = handler.getMessages();
 		String msg = ((TestMessageHandler.TestMessage)msgs.get(0)).getContainedMessage().getMessage();
 		String exp = "-outxml/-outxmlfile option ignored because resource already exists:";
 		assertTrue("Expected message to start with : " + exp + " but found message " + msg,msg.startsWith(exp));

ajde.core/testsrc/org/aspectj/ajde/core/tests/ResourceCopyTests.java

 	}
 
 	public void testInjarsToBin() {
-		Set injars = new HashSet();
+		Set<File> injars = new HashSet<File>();
 		File injar1 = openFile(injar1Name);
 		injars.add(injar1);
 		compilerConfig.setInpath(injars);

asm/src/org/aspectj/asm/AsmManager.java

 				((AspectJElementHierarchy) hierarchy).setAsmManager(this);
 				hierarchyReadOK = true;
 				mapper = (RelationshipMap) s.readObject();
+				s.close();
 			}
 		} catch (FileNotFoundException fnfe) {
 			// That is OK
 			Set<String> sourcesToRemove = new HashSet<String>();
 			Set<String> nonExistingHandles = new HashSet<String>(); // Cache of handles that we
 			// *know* are invalid
-			int srchandlecounter = 0;
-			int tgthandlecounter = 0;
+//			int srchandlecounter = 0;
+//			int tgthandlecounter = 0;
 
 			// Iterate over the source handles in the relationships map
 			Set<String> keyset = mapper.getEntries(); // These are source handles
 			for (String hid : keyset) {
-				srchandlecounter++;
+//				srchandlecounter++;
 
 				// Do we already know this handle points to nowhere?
 				if (nonExistingHandles.contains(hid)) {
 							// Iterate through the targets for this relationship
 							for (Iterator<String> targetIter = targets.iterator(); targetIter.hasNext();) {
 								String targethid = targetIter.next();
-								tgthandlecounter++;
+//								tgthandlecounter++;
 								// Do we already know it doesn't exist?
 								if (nonExistingHandles.contains(targethid)) {
 									if (dumpDeltaProcessing) {

asm/src/org/aspectj/asm/internal/ProgramElement.java

 		}
 	}
 
+	@SuppressWarnings("unchecked")
 	public Map<String, List<String>> getDeclareParentsMap() {
 		Map<String, List<String>> s = (Map<String, List<String>>) kvpairs.get("declareparentsmap");
 		return s;

runtime/testsrc/org/aspectj/runtime/reflect/RuntimePerformanceTest.java

 	private static final long TIMEOUT = 10000;
 	private static final long ITERATIONS = 1000000;
 	private static final long WARMUP_ITERATIONS = 10000;
-	private static final long EXPECTED_RATIO = 10;
+	private static final long EXPECTED_RATIO = 8;
 	private static final Factory factory = new Factory("RutimePerformanceTest.java",RuntimePerformanceTest.class);
 
 	private boolean savedUseCaches;