]> source.dussan.org Git - aspectj.git/commitdiff
Weaken Collection declarations
authorLars Grefer <eclipse@larsgrefer.de>
Sat, 15 Aug 2020 15:01:46 +0000 (17:01 +0200)
committerLars Grefer <eclipse@larsgrefer.de>
Sat, 15 Aug 2020 15:01:46 +0000 (17:01 +0200)
Reports on declarations of Collection variables made by using the collection class as the type, rather than an appropriate interface.

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>
80 files changed:
ajbrowser/src/main/java/org/aspectj/tools/ajbrowser/ui/EditorManager.java
ajde.core/src/main/java/org/aspectj/ajde/core/internal/AjdeCoreBuildManager.java
ajde/src/main/java/org/aspectj/ajde/ui/StructureViewManager.java
ajde/src/main/java/org/aspectj/ajde/ui/swing/PointcutWizard.java
ajdoc/src/main/java/org/aspectj/tools/ajdoc/JavadocRunner.java
ajdoc/src/main/java/org/aspectj/tools/ajdoc/Main.java
ajdoc/src/main/java/org/aspectj/tools/ajdoc/StubFileGenerator.java
asm/src/main/java/org/aspectj/asm/AsmManager.java
asm/src/main/java/org/aspectj/asm/IHierarchy.java
asm/src/main/java/org/aspectj/asm/internal/AspectJElementHierarchy.java
bcel-builder/src/main/java/org/aspectj/apache/bcel/generic/FieldGenOrMethodGen.java
bcel-builder/src/main/java/org/aspectj/apache/bcel/generic/InstructionList.java
bcel-builder/src/main/java/org/aspectj/apache/bcel/generic/MethodGen.java
bcel-builder/src/main/java/org/aspectj/apache/bcel/util/ClassPath.java
bcel-builder/src/test/java/org/aspectj/apache/bcel/classfile/tests/AnnotationGenTest.java
bcel-builder/src/test/java/org/aspectj/apache/bcel/classfile/tests/GenericSignatureParsingTest.java
bridge/src/main/java/org/aspectj/bridge/MessageHandler.java
bridge/src/main/java/org/aspectj/bridge/MessageUtil.java
bridge/src/main/java/org/aspectj/bridge/context/CompilationAndWeavingContext.java
build/src/main/java/org/aspectj/internal/tools/ant/taskdefs/Checklics.java
build/src/main/java/org/aspectj/internal/tools/build/Builder.java
build/src/main/java/org/aspectj/internal/tools/build/Module.java
build/src/main/java/org/aspectj/internal/tools/build/Result.java
build/src/main/java/org/aspectj/internal/tools/build/SampleGatherer.java
build/src/main/java/org/aspectj/internal/tools/build/Util.java
build/src/test/java/org/aspectj/build/BuildModuleTests.java
build/src/test/java/org/aspectj/internal/build/BuildModuleTest.java
build/src/test/java/org/aspectj/internal/build/ModulesTest.java
org.aspectj.ajdt.core/src/main/java/org/aspectj/ajdt/ajc/BuildArgParser.java
org.aspectj.ajdt.core/src/main/java/org/aspectj/ajdt/internal/compiler/ast/AstUtil.java
org.aspectj.ajdt.core/src/main/java/org/aspectj/ajdt/internal/core/builder/AjBuildConfig.java
org.aspectj.ajdt.core/src/test/java/org/aspectj/ajdt/ajc/AjdtCommandTestCase.java
org.aspectj.ajdt.core/src/test/java/org/aspectj/ajdt/internal/compiler/batch/IncrementalCase.java
org.aspectj.ajdt.core/src/test/java/org/aspectj/tools/ajc/ASTVisitorTest.java
org.aspectj.ajdt.core/src/test/java/org/aspectj/tools/ajc/AjcTestCase.java
org.aspectj.ajdt.core/src/test/java/org/aspectj/tools/ajc/MainTest.java
org.aspectj.matcher/src/main/java/org/aspectj/weaver/CrosscuttingMembers.java
org.aspectj.matcher/src/main/java/org/aspectj/weaver/ResolvedType.java
taskdefs/src/main/java/org/aspectj/tools/ant/taskdefs/AjcTask.java
taskdefs/src/test/java/org/aspectj/tools/ant/taskdefs/Ajc11CompilerAdapterTest.java
testing-client/src/main/java/org/aspectj/testing/Tester.java
testing-client/src/test/java/org/aspectj/testing/TesterTest.java
testing-drivers/src/test/java/org/aspectj/testing/drivers/Harness.java
testing/src/test/java/org/aspectj/testing/harness/bridge/AbstractRunSpec.java
testing/src/test/java/org/aspectj/testing/harness/bridge/AjcSpecTest.java
testing/src/test/java/org/aspectj/testing/harness/bridge/AjcTest.java
testing/src/test/java/org/aspectj/testing/harness/bridge/CompilerRun.java
testing/src/test/java/org/aspectj/testing/harness/bridge/DirChanges.java
testing/src/test/java/org/aspectj/testing/harness/bridge/FlatSuiteReader.java
testing/src/test/java/org/aspectj/testing/harness/bridge/Globals.java
testing/src/test/java/org/aspectj/testing/harness/bridge/IncCompilerRun.java
testing/src/test/java/org/aspectj/testing/harness/bridge/Validator.java
testing/src/test/java/org/aspectj/testing/run/RunListeners.java
testing/src/test/java/org/aspectj/testing/taskdefs/AjcTaskCompileCommandTest.java
testing/src/test/java/org/aspectj/testing/util/Diffs.java
testing/src/test/java/org/aspectj/testing/util/FileUtil.java
testing/src/test/java/org/aspectj/testing/util/LangUtil.java
testing/src/test/java/org/aspectj/testing/util/LangUtilTest.java
testing/src/test/java/org/aspectj/testing/util/LinkCheck.java
testing/src/test/java/org/aspectj/testing/util/MessageUtilTest.java
testing/src/test/java/org/aspectj/testing/util/TestClassLoader.java
testing/src/test/java/org/aspectj/testing/util/TestDiffs.java
testing/src/test/java/org/aspectj/testing/util/TestDiffsTest.java
testing/src/test/java/org/aspectj/testing/util/options/Option.java
testing/src/test/java/org/aspectj/testing/util/options/Options.java
testing/src/test/java/org/aspectj/testing/xml/AjcSpecXmlReaderTest.java
testing/src/test/java/org/aspectj/testing/xml/MessageListXmlReaderTest.java
testing/src/test/java/org/aspectj/testing/xml/SoftMessage.java
tests/src/test/java/org/aspectj/systemtest/incremental/tools/AnnotationProcessingTests.java
tests/src/test/java/org/aspectj/systemtest/incremental/tools/MultiProjTestCompilerConfiguration.java
tests/src/test/java/org/aspectj/systemtest/incremental/tools/MultiProjectIncrementalTests.java
util/src/main/java/org/aspectj/util/FileUtil.java
util/src/main/java/org/aspectj/util/LangUtil.java
util/src/main/java/org/aspectj/util/UtilClassLoader.java
util/src/test/java/org/aspectj/util/FileUtilTest.java
weaver/src/main/java/org/aspectj/weaver/bcel/BcelWeaver.java
weaver/src/main/java/org/aspectj/weaver/bcel/ClassPathManager.java
weaver/src/main/java/org/aspectj/weaver/bcel/LazyMethodGen.java
weaver/src/main/java/org/aspectj/weaver/loadtime/definition/SimpleAOPParser.java
weaver/src/test/java/org/aspectj/weaver/ReferenceTypeTestCase.java

index b737cf07e9be09c1318a1684d57950452afbeac5..249992dd72cb03599d61ae43ce5abde4065cc152 100644 (file)
@@ -17,6 +17,7 @@ import java.awt.BorderLayout;
 import java.awt.event.KeyEvent;
 import java.io.IOException;
 import java.util.ArrayList;
+import java.util.List;
 import java.util.Vector;
 
 import javax.swing.Box;
@@ -43,7 +44,7 @@ public class EditorManager {
 
        private EditorAdapter editor = null;
        private BasicEditor basicEditor = null;
-       private ArrayList<EditorListener> editorListeners = new ArrayList<>();
+       private List<EditorListener> editorListeners = new ArrayList<>();
        private Vector<EditorAdapter> editors = new Vector<>();
        private JPanel editor_panel = null;
        private Box editors_box = Box.createVerticalBox();
index 7aec208092963a8d954ef95b7ef82c5286ec48ef..e9f3a1b8632c3844288288508313a2f0877b3ac9 100644 (file)
@@ -278,7 +278,7 @@ public class AjdeCoreBuildManager {
                                }
                        }
                        if (0 < toAdd.size()) {
-                               ArrayList<String> both = new ArrayList<>(configClasspath.size() + toAdd.size());
+                               List<String> both = new ArrayList<>(configClasspath.size() + toAdd.size());
                                both.addAll(configClasspath);
                                both.addAll(toAdd);
                                config.setClasspath(both);
index 9b23439e4fdf55e576a7b35a8667fb1d51a17cee..8f60ff824dfa4a36900dd11c898d6bb401f9d11c 100644 (file)
@@ -36,7 +36,7 @@ public class StructureViewManager {
        // private String buildConfigFilePath = null;
 
        private final NavigationHistoryModel historyModel = new NavigationHistoryModel();
-       private final ArrayList structureViews = new ArrayList();
+       private final List structureViews = new ArrayList();
        private FileStructureView defaultFileView = null;
 
        private static final StructureViewProperties DEFAULT_VIEW_PROPERTIES;
index fb461d1326a242fa1544335bf51e534d7634e386..f80ff76e67de1138d834c8909c7bad89b2eec8b4 100644 (file)
@@ -17,6 +17,7 @@ package org.aspectj.ajde.ui.swing;
 import java.awt.BorderLayout;
 //import java.awt.event.ActionEvent;
 import java.util.ArrayList;
+import java.util.List;
 
 import javax.swing.JButton;
 import javax.swing.JCheckBox;
@@ -55,7 +56,7 @@ class PointcutWizard extends JFrame {
 
     public PointcutWizard(java.util.List signatures) {
  //       this.signatures = signatures;
-        ArrayList views = new ArrayList();
+        List views = new ArrayList();
         views.add(StructureViewProperties.Hierarchy.INHERITANCE);
 //        typeTreeView = new BrowserViewPanel(AjdeUIManager.getDefault().getIconRegistry(), views, StructureViewProperties.Hierarchy.INHERITANCE);
         
index adf22bf1223c84a9cba573a96a2fcebf2a088a07..e1bbd9ad45a92f464e42d8658db7fc8d16a6b888 100644 (file)
@@ -28,7 +28,7 @@ import javax.tools.ToolProvider;
  */
 class JavadocRunner {
 
-       public static void callJavadocViaToolProvider(Vector<String> options, List<String> files) {
+       public static void callJavadocViaToolProvider(Iterable<String> options, List<String> files) {
                DocumentationTool doctool = ToolProvider.getSystemDocumentationTool();
                StandardJavaFileManager fm = doctool.getStandardFileManager(null, null, null);
                Iterable<? extends JavaFileObject> jfos = fm.getJavaFileObjects(files.toArray(new String[0]));
index 57b7bce6ed64c6a2807ccf9918b899a879123a89..e65c8901f29de331cff89951e686cc4fcc13812a 100644 (file)
@@ -157,7 +157,7 @@ public class Main implements Config {
         * package-summary properly.
         */
        private static void packageHTML(AsmManager model, File[] inputFiles) throws IOException {
-               ArrayList<String> dirList = new ArrayList<>();
+               List<String> dirList = new ArrayList<>();
                for (File inputFile : inputFiles) {
                        String packageName = StructureUtil.getPackageDeclarationFromFile(model, inputFile);
                        // Only copy the package.html file once.
index 4b9301ce817afba06b63f517889ec38b33a2836a..c68123aba27779bb06b12d10c3b635fa710558b8 100644 (file)
@@ -22,6 +22,7 @@ import java.io.PrintWriter;
 import java.util.Hashtable;
 import java.util.Iterator;
 import java.util.List;
+import java.util.Map;
 
 import org.aspectj.asm.AsmManager;
 import org.aspectj.asm.IProgramElement;
@@ -32,7 +33,7 @@ import org.aspectj.util.FileUtil;
  */
 class StubFileGenerator {
 
-       static Hashtable declIDTable = null;
+       static Map declIDTable = null;
 
        static void doFiles(AsmManager model, Hashtable table, File[] inputFiles, File[] signatureFiles) throws DocException {
                declIDTable = table;
index dc766eb7f247ba9c73565a671edccf42a676ad13..5fcaf3cf62a50597c2a327b3fa9d189f7206b000 100644 (file)
@@ -155,7 +155,7 @@ public class AsmManager implements IStructureModel {
                        return null;
                } else {
                        IProgramElement fileNode = node;
-                       ArrayList<IProgramElement> peNodes = new ArrayList<>();
+                       List<IProgramElement> peNodes = new ArrayList<>();
                        getAllStructureChildren(fileNode, peNodes, showSubMember, showMemberAndType);
                        for (IProgramElement peNode : peNodes) {
                                List<IProgramElement> entries = new ArrayList<>();
index 6674cdaea7d333b34810ab32df8e52b744017147..80ed2b55136d6459e4d22e6a383e4e015b49cf84 100644 (file)
@@ -39,7 +39,11 @@ public interface IHierarchy extends Serializable {
 
        public boolean removeFromFileMap(String canonicalFilePath);
 
-       public void setFileMap(HashMap<String, IProgramElement> fileMap);
+       public void setFileMap(Map<String, IProgramElement> fileMap);
+
+       public default void setFileMap(HashMap<String, IProgramElement> fileMap) {
+               setFileMap((Map<String, IProgramElement>) fileMap);
+       }
 
        public Object findInFileMap(Object key);
 
index 869affd91f3de6ca51c1e5c00de95e75ec0bb2d3..c524c554a8bda02ec6aadaf8efa1991f41d52679 100644 (file)
@@ -96,7 +96,7 @@ public class AspectJElementHierarchy implements IHierarchy {
                return fileMap.remove(canonicalFilePath) != null;
        }
 
-       public void setFileMap(HashMap<String, IProgramElement> fileMap) {
+       public void setFileMap(Map<String, IProgramElement> fileMap) {
                this.fileMap = fileMap;
        }
 
index c8e1fe2fb4411adf0aff63751a5c3793124a49dc..5b4256685bef4c843416f6061fe87eda87501b71 100644 (file)
@@ -76,8 +76,8 @@ public abstract class FieldGenOrMethodGen extends Modifiers {
        protected String name;
        protected Type type;
        protected ConstantPool cp;
-       private ArrayList<Attribute> attributeList = new ArrayList<>();
-       protected ArrayList<AnnotationGen> annotationList = new ArrayList<>();
+       private List<Attribute> attributeList = new ArrayList<>();
+       protected List<AnnotationGen> annotationList = new ArrayList<>();
 
        protected FieldGenOrMethodGen() {
        }
index 156adf0ad149487d251eb6ba4f532a81ccd67333..7938218bc45b65d113ec87b23955a40ad48e5cd2 100644 (file)
@@ -58,10 +58,7 @@ import java.io.ByteArrayOutputStream;
 import java.io.DataOutputStream;
 import java.io.IOException;
 import java.io.Serializable;
-import java.util.ArrayList;
-import java.util.HashMap;
-import java.util.Iterator;
-import java.util.Set;
+import java.util.*;
 
 import org.aspectj.apache.bcel.Constants;
 import org.aspectj.apache.bcel.classfile.Constant;
@@ -690,7 +687,7 @@ public class InstructionList implements Serializable {
                        return;
                }
 
-               ArrayList<InstructionHandle> target_vec = new ArrayList<>();
+               List<InstructionHandle> target_vec = new ArrayList<>();
 
                for (InstructionHandle ih = first; ih != null; ih = ih.next) {
                        ih.getInstruction().dispose(); // e.g. BranchInstructions release their targets
@@ -1003,7 +1000,7 @@ public class InstructionList implements Serializable {
         */
        public Instruction[] getInstructions() {
                ByteSequence bytes = new ByteSequence(getByteCode());
-               ArrayList<Instruction> instructions = new ArrayList<>();
+               List<Instruction> instructions = new ArrayList<>();
 
                try {
                        while (bytes.available() > 0) {
index cf8c068347996a0674a68f64e7ee40beb78c8079..453fe7c1f067ea5999c7305363c2282ad19ba818 100644 (file)
@@ -54,11 +54,7 @@ package org.aspectj.apache.bcel.generic;
  * <http://www.apache.org/>.
  */
 
-import java.util.ArrayList;
-import java.util.Hashtable;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Stack;
+import java.util.*;
 
 import org.aspectj.apache.bcel.Constants;
 import org.aspectj.apache.bcel.classfile.Attribute;
@@ -103,11 +99,11 @@ public class MethodGen extends FieldGenOrMethodGen {
 
        private int highestLineNumber = 0;
 
-       private ArrayList<LocalVariableGen> localVariablesList = new ArrayList<>();
-       private ArrayList<LineNumberGen> lineNumbersList = new ArrayList<>();
+       private List<LocalVariableGen> localVariablesList = new ArrayList<>();
+       private List<LineNumberGen> lineNumbersList = new ArrayList<>();
        private ArrayList<CodeExceptionGen> exceptionsList = new ArrayList<>();
        private ArrayList<String> exceptionsThrown = new ArrayList<>();
-       private ArrayList<Attribute> codeAttributesList = new ArrayList<>();
+       private List<Attribute> codeAttributesList = new ArrayList<>();
        private List<AnnotationGen>[] param_annotations; // Array of lists containing AnnotationGen objects
        private boolean hasParameterAnnotations = false;
        private boolean haveUnpackedParameterAnnotations = false;
@@ -934,7 +930,7 @@ public class MethodGen extends FieldGenOrMethodGen {
 
        static final class BranchStack {
                Stack<BranchTarget> branchTargets = new Stack<>();
-               Hashtable<InstructionHandle, BranchTarget> visitedTargets = new Hashtable<>();
+               Map<InstructionHandle, BranchTarget> visitedTargets = new Hashtable<>();
 
                public void push(InstructionHandle target, int stackDepth) {
                        if (visited(target)) {
index 1798005403313af776b9b805410f2dcfcd495ec4..678abb1dcddddb1c09896cca7cb1858caf88eddf 100644 (file)
@@ -69,11 +69,7 @@ import java.nio.file.Path;
 import java.nio.file.SimpleFileVisitor;
 import java.nio.file.attribute.BasicFileAttributeView;
 import java.nio.file.attribute.BasicFileAttributes;
-import java.util.ArrayList;
-import java.util.HashMap;
-import java.util.Iterator;
-import java.util.Map;
-import java.util.StringTokenizer;
+import java.util.*;
 import java.util.zip.ZipEntry;
 import java.util.zip.ZipFile;
 
@@ -106,7 +102,7 @@ public class ClassPath implements Serializable {
        public ClassPath(String class_path) {
                this.class_path = class_path;
 
-               ArrayList<PathEntry> vec = new ArrayList<>();
+               List<PathEntry> vec = new ArrayList<>();
 
                for (StringTokenizer tok = new StringTokenizer(class_path, System.getProperty("path.separator")); tok
                                .hasMoreTokens();) {
@@ -167,7 +163,7 @@ public class ClassPath implements Serializable {
                return false;
        }
 
-       private static final void getPathComponents(String path, ArrayList<String> list) {
+       private static final void getPathComponents(String path, List<String> list) {
                if (path != null) {
                        StringTokenizer tok = new StringTokenizer(path, File.pathSeparator);
 
index d684144eb2e9e161c5ed8bb4bd3cd0f3c45fca41..84f9d95321bbeef3be32069307da1c84acc2b312 100644 (file)
@@ -97,7 +97,7 @@ public class AnnotationGenTest extends BcelTestCase {
                // Build a RV annotation of type 'SimpleAnnotation' with 'id=4' as the only value :)
                AnnotationGen a = new AnnotationGen(t, elements, true, cp);
 
-               Vector<AnnotationGen> v = new Vector<>();
+               List<AnnotationGen> v = new Vector<>();
                v.add(a);
                Collection<RuntimeAnnos> attributes = Utility.getAnnotationAttributes(cp, v);
                boolean foundRV = false;
@@ -113,7 +113,7 @@ public class AnnotationGenTest extends BcelTestCase {
                // Build a RIV annotation of type 'SimpleAnnotation' with 'id=4' as the only value :)
                AnnotationGen a2 = new AnnotationGen(t, elements, false, cp);
 
-               Vector<AnnotationGen> v2 = new Vector<>();
+               List<AnnotationGen> v2 = new Vector<>();
                v2.add(a2);
                Collection<RuntimeAnnos> attributes2 = Utility.getAnnotationAttributes(cp, v2);
                boolean foundRIV = false;
index 255dc77ffcee3c0945e235344bc4a475bf17d813..c97e12678b77f55c026460a63db2cc3a99ad96e2 100644 (file)
@@ -12,6 +12,7 @@
 package org.aspectj.apache.bcel.classfile.tests;
 
 import java.util.ArrayList;
+import java.util.List;
 
 import org.aspectj.apache.bcel.Constants;
 import org.aspectj.apache.bcel.classfile.Attribute;
@@ -415,7 +416,7 @@ public class GenericSignatureParsingTest extends BcelTestCase {
           * @return              Array of argument types
           */
          public static final String[] methodSignatureArgumentTypes(String signature,boolean chopit) throws ClassFormatException {
-           ArrayList<String> vec = new ArrayList<>();
+           List<String> vec = new ArrayList<>();
            int       index;
            String[]  types;
        
index 23c2ef3b2628f48d62284b0f3ec0b1eae0c588b2..95bcf029db0378baaa3de44e7c223465e56529ec 100644 (file)
@@ -31,7 +31,7 @@ public class MessageHandler implements IMessageHolder {
        /** messages accumulated */
        protected final ArrayList<IMessage> messages;
        /** kinds of messages to be ignored */
-       protected final ArrayList<IMessage.Kind> ignoring;
+       protected final List<IMessage.Kind> ignoring;
        /** result of handleMessage(..) for messages not accumulated (ignored) */
        protected boolean handleMessageResult;
        /** listener which can halt processing by returning true */
index d55ac3c3b4f35449a04e56d81af14d89cabfd68c..0ebe113e0af5b3edf4c7b5f0f3edccbb612618f3 100644 (file)
@@ -409,7 +409,7 @@ public class MessageUtil {
                if (null == kind) {
                        return messages;
                }
-               ArrayList<IMessage> result = new ArrayList<>();
+               List<IMessage> result = new ArrayList<>();
                for (IMessage message : messages) {
                        if (kind == message.getKind()) {
                                result.add(message);
index 2ae13a27915850b1459e8f9f6d5646cea9c01e89..e1ea09a99e5ab497dc61ce2c6768bc82dffed826 100644 (file)
@@ -130,7 +130,7 @@ public class CompilationAndWeavingContext {
         * Returns a string description of what the compiler/weaver is currently doing
         */
        public static String getCurrentContext() {
-               Stack<ContextStackEntry> contextStack = getContextStack();
+               Iterable<ContextStackEntry> contextStack = getContextStack();
                Stack<String> explanationStack = new Stack<>();
                for (ContextStackEntry entry : contextStack) {
                        Object data = entry.getData();
index 5d592243ce5ee6f9f9618fe71557d1b06ca0bd7f..e343c9d48cdcd71a3fcd4089531bc9e6bd4ef37d 100644 (file)
@@ -654,7 +654,7 @@ class Header {
         *
         * @return true if this line has end-of-comment
         */
-       private static boolean checkLine(String line, ArrayList<String> years) {
+       private static boolean checkLine(String line, List<String> years) {
                if ((null == line) || (0 == line.length())) {
                        return false;
                }
index ac13870c853cab6d8ab7023544c76c3faacf2c32..fad8b560a8cac882a039dfd26c741cb3f5f8e3a8 100644 (file)
@@ -181,7 +181,7 @@ public abstract class Builder {
 
     private final File tempDir;
 
-    private final ArrayList tempFiles;
+    private final List tempFiles;
 
     private final boolean useEclipseCompiles;
 
@@ -275,7 +275,7 @@ public abstract class Builder {
             return buildProduct(buildSpec);
         }
         Result result = specifyResultFor(buildSpec);
-        ArrayList<String> errors = new ArrayList<>();
+        List<String> errors = new ArrayList<>();
         try {
             return buildAll(result, errors);
         } finally {
@@ -340,7 +340,7 @@ public abstract class Builder {
      */
     protected final boolean buildAll(Result result, List<String> errors) {
         Result[] buildList = skipUptodate(getAntecedantResults(result));
-        ArrayList<String> doneList = new ArrayList<>();
+        List<String> doneList = new ArrayList<>();
         if ((null != buildList) && (0 < buildList.length)) {
             if (isLogging()) {
                 handler.log("modules to build: " + Arrays.asList(buildList));
@@ -525,7 +525,7 @@ public abstract class Builder {
     }
 
     protected final boolean buildProductModule(ProductModule module) {
-        ArrayList<String> errors = new ArrayList<>();
+        List<String> errors = new ArrayList<>();
         try {
             Kind productKind = Result.kind(Result.NORMAL, Result.ASSEMBLE);
             Result result = module.module.getResult(productKind);
@@ -543,7 +543,7 @@ public abstract class Builder {
      * deliverables.
      */
     protected ProductModule[] discoverModules(File productDir, Modules modules) {
-        final ArrayList<File> found = new ArrayList<>();
+        final List<File> found = new ArrayList<>();
         FileFilter filter = new FileFilter() {// empty jar files
             public boolean accept(File file) {
                 if ((null != file) && file.canRead()
index 3f34b3f5ea876ed46ff5f56ca5efc19eec0cd609..9d10e4743d8168b5592685f5d915ba5e302760ab 100644 (file)
@@ -69,7 +69,7 @@ public class Module {
 
     /** @return all source files under srcDir */
     private static Iterator<File> sourceFiles(File srcDir) {
-        ArrayList<File> result = new ArrayList<>();
+        List<File> result = new ArrayList<>();
         sourceFiles(srcDir, result);
         return result.iterator();
     }
index 563a6932e41457701243c3c183e6dd69d578d95a..22b92c8afb4ea9e576d6e0de7bffc2b7cd85b7f0 100644 (file)
@@ -217,7 +217,7 @@ public class Result {
 
     /** @return List (File) of jar's required */
     public List<File> findJarRequirements() {
-        ArrayList<File> result = new ArrayList<>();
+        List<File> result = new ArrayList<>();
         Module.doFindJarRequirements(this, result);
         return result;
     }
index bf9b9aab90b4dfe49eb23b93d4b14db33c8cfe25..9030bbf77081743805af9aa3a915e0cdd2b84050 100644 (file)
@@ -419,7 +419,7 @@ class Samples {
     }
 
     List<Sample> getSortedSamples(Comparator<Sample> comparer) {
-               ArrayList<Sample> result = new ArrayList<>(samples);
+               List<Sample> result = new ArrayList<>(samples);
         result.sort(comparer);
         return result;
     }
index dd393ce45ca915f2e35640f445dccd54a3af4200..cd7d6623f1b302c025dd65dd69392e313adfd344 100644 (file)
@@ -357,7 +357,7 @@ public class Util {
         /** unmodifiable list of all valid OSGIBundle Name's */
         public static final List<Name> NAMES;
         static {
-            ArrayList<Name> names = new ArrayList<>();
+            List<Name> names = new ArrayList<>();
             names.add(BUNDLE_NAME);
             names.add(BUNDLE_SYMBOLIC_NAME);
             names.add(BUNDLE_VERSION);
index f4aaa570861fb1cb46a7052a8dac61ab7dbb6d2b..46f5b15a04ee54b3a309640994b010e416de637b 100644 (file)
@@ -225,7 +225,7 @@ public class BuildModuleTests extends TestCase {
      */
     static class UnknownFileCheck implements FileFilter {
         private static final UnknownFileCheck SINGLETON = new UnknownFileCheck();
-        private static final ArrayList<String> STATIC_ERRORS = new ArrayList<>();
+        private static final List<String> STATIC_ERRORS = new ArrayList<>();
         // Builder.BINARY_SOURCE_PATTERN and Builder.RESOURCE_PATTERN
         public static final List<String> KNOWN_SUFFIXES;
 
index 7693ddb11edc78cb2e1fd19fd781f3ce7afefd87..33dac2a0af3c62fda03c41f3104e97af9384877c 100644 (file)
@@ -16,10 +16,7 @@ package org.aspectj.internal.build;
 
 import java.io.File;
 import java.io.IOException;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Enumeration;
-import java.util.Iterator;
+import java.util.*;
 import java.util.zip.ZipEntry;
 import java.util.zip.ZipException;
 import java.util.zip.ZipFile;
@@ -80,7 +77,7 @@ public class BuildModuleTest extends TestCase {
         }
     }
         
-    ArrayList<File> tempFiles = new ArrayList<>();
+    List<File> tempFiles = new ArrayList<>();
     private File jarDir;
     private boolean deleteJars;
     boolean building;  // must be enabled for tests to run
@@ -343,7 +340,7 @@ public class BuildModuleTest extends TestCase {
         try {
             zipFile = new ZipFile(weaverAllJar);
             Enumeration e = zipFile.entries();
-            ArrayList<String> entryNames = new ArrayList<>();
+            List<String> entryNames = new ArrayList<>();
             while (e.hasMoreElements()) {
                 ZipEntry entry = (ZipEntry) e.nextElement();
                 String name = entry.getName();
index 494084e0c9e3500e232c47c21451b354f7b46bb3..6273556a5f5c9c16acf754130d0b8d219677d14f 100644 (file)
@@ -72,7 +72,7 @@ public class ModulesTest extends TestCase {
         }
     }
 
-    ArrayList<File> tempFiles = new ArrayList<>();
+    List<File> tempFiles = new ArrayList<>();
      
        public ModulesTest(String name) {
                super(name);
@@ -101,7 +101,7 @@ public class ModulesTest extends TestCase {
     }
       
     public void testAllModulesCreation() {
-        ArrayList<Module> badModules = new ArrayList<>();
+        List<Module> badModules = new ArrayList<>();
         for (String name: MODULE_NAMES) {
             File dir = new File(BASE_DIR, name);
             if (dir.isDirectory()) {
index c309a3a5aa6422e3e06cb159c7af44366072d440..8790e964bacde2c5d47f6a0187fbe9dfa02d768a 100644 (file)
@@ -903,7 +903,7 @@ public class BuildArgParser extends Main {
                        unparsedArgs.add(nextArg.getValue());
                }
 
-               private int indexOf(LinkedList<Arg> args, String arg) {
+               private int indexOf(Iterable<Arg> args, String arg) {
                        int index = 0;
                        for (Arg argument : args) {
                                if (arg.equals(argument.getValue())) {
index 6dc405653f1884d3e8ff9ee9093ea686c32c679c..04693fd867737d63370d0b4c1949a539f6ce10f0 100644 (file)
@@ -217,7 +217,7 @@ public class AstUtil {
        }
 
        public static List getAjSyntheticAttribute() {
-               ArrayList ret = new ArrayList(1);
+               List ret = new ArrayList(1);
                ret.add(new EclipseAttributeAdapter(new AjAttribute.AjSynthetic()));
                return ret;
        }
index 4fb176e881658bd917409bfbe7967505b221202d..35a71fd109a658308740a1d819eeedbe1bf2c09f 100644 (file)
@@ -69,10 +69,10 @@ public class AjBuildConfig implements CompilerConfigurationChangeFlags {
        private List<String> classpath = new ArrayList<>();
        private List<String> modulepath = new ArrayList<>();
        // Expensive to compute (searching modules, parsing module-info)
-       private ArrayList<Classpath> modulepathClasspathEntries = null;
+       private Collection<Classpath> modulepathClasspathEntries = null;
        private List<String> modulesourcepath = new ArrayList<>();
        // Expensive to compute (searching modules, parsing module-info)
-       private ArrayList<Classpath> modulesourcepathClasspathEntries = null;
+       private Collection<Classpath> modulesourcepathClasspathEntries = null;
        private Classpath[] checkedClasspaths = null;
        private List<String> bootclasspath = new ArrayList<>();
        private List<String> cpElementsWithModifiedContents = new ArrayList<>();
index 57c2f8e4c8088abd754fbced391dd10e0636d360..95bd5c4c6a7a75930b298e6d7511341343e6c87c 100644 (file)
@@ -19,6 +19,7 @@ import java.io.IOException;
 import java.io.PrintStream;
 import java.io.PrintWriter;
 import java.util.ArrayList;
+import java.util.List;
 import java.util.ListIterator;
 
 import junit.framework.TestCase;
@@ -38,7 +39,7 @@ import org.aspectj.util.FileUtil;
  */
 public class AjdtCommandTestCase extends TestCase {
 
-       private ArrayList tempFiles = new ArrayList();
+       private List tempFiles = new ArrayList();
        private StreamPrintWriter outputWriter = new StreamPrintWriter(new PrintWriter(System.out));
        // private AjdtCommand command = new AjdtCommand();
        private MessageWriter messageWriter = new MessageWriter(outputWriter, false);
index 09ecdb34b3e61b79c2dfd79d4a3048536825a53b..0d84dfc8f7ee05deff0df1cf4512af276a70f6ed 100644 (file)
@@ -84,7 +84,7 @@ public class IncrementalCase { // XXX NOT bound to junit - bridge tests?
                if (null == targetClasses) {
                        return false;
                }
-               final ArrayList<File> files = new ArrayList<>();
+               final List<File> files = new ArrayList<>();
                final FileFilter collector = new FileFilter() {
                        @Override
                        public boolean accept(File file) {
@@ -94,7 +94,7 @@ public class IncrementalCase { // XXX NOT bound to junit - bridge tests?
                final ICommand compiler =
                        ReflectionFactory.makeCommand(cname, handler);
 
-               ArrayList recompiled = null;
+               List recompiled = null;
                boolean result = true;
 
                final String toSuffix = ".java";
@@ -257,7 +257,7 @@ public class IncrementalCase { // XXX NOT bound to junit - bridge tests?
 
 
     List<String> normalizeFilenames(String[] ra) { // XXX util
-        ArrayList<String> result = new ArrayList<>();
+        List<String> result = new ArrayList<>();
         if (null != ra) {
                        for (String s : ra) {
                                result.add(normalizeFilename(s));
@@ -271,7 +271,7 @@ public class IncrementalCase { // XXX NOT bound to junit - bridge tests?
 
     /** @param list the List of File */
     List<String> normalizeFilenames(List<File> list) { // XXX util
-        ArrayList<String> result = new ArrayList<>();
+        List<String> result = new ArrayList<>();
         for (File file: list) {
 //        for (Iterator<?> iter = list.iterator(); iter.hasNext();) {
                        result.add(normalizeFilename(file.getPath()));                  
index de8960e7341fe183c47863e4f0ef3eb8faa7d321..b38e8166c675a6fd0bb143c319582981ea4807fe 100644 (file)
@@ -421,7 +421,7 @@ public class ASTVisitorTest extends TestCase {
        
        private void checkJLS3(String source, String expectedOutput) {
                ASTParser parser = ASTParser.newParser(AST.JLS3);
-               HashMap<String,String> options = new HashMap<>();
+               Map<String,String> options = new HashMap<>();
                options.put(JavaCore.COMPILER_SOURCE, JavaCore.VERSION_1_5);
                parser.setCompilerOptions(options);
                parser.setSource(source.toCharArray());
index 518f944c8c013adf87a4d7271b1d86a7032fdd41..91fbc9b8662b6d7e215d8f4a6428bb3e9166f252 100644 (file)
@@ -787,7 +787,7 @@ public abstract class AjcTestCase extends TestCase {
 
        private List<String >tokenizeCommand(String command) {
                StringTokenizer st = new StringTokenizer(command," ", false);
-               ArrayList<String> arguments = new ArrayList<>();
+               List<String> arguments = new ArrayList<>();
                while(st.hasMoreElements()){
                        String nextToken =st.nextToken();
                        arguments.add(nextToken);
index 7da3aa6474c49a72aad3da873158986f3925ac18..ddcd1046a818cd7a6de349a8b4b11d6b4cad8924 100644 (file)
@@ -13,6 +13,7 @@
 package org.aspectj.tools.ajc;
 
 import java.util.ArrayList;
+import java.util.List;
 import java.util.ResourceBundle;
 
 import org.aspectj.bridge.AbortException;
@@ -23,7 +24,7 @@ import org.aspectj.bridge.AbortException;
 public class MainTest extends AjcTestCase {
        
     public void testMainbare() {
-        ArrayList<String> list = new ArrayList<>();
+        List<String> list = new ArrayList<>();
 // Usage now printed by Eclipse compiler so doesn't appear here in our message list
 //        Main.bareMain(new String[] {"-help"}, false, list, null, null, null);
 //        assertTrue(1 == list.size());
index d380e988e6663f816a3faa18413eb099de4aef2f..e71850c0580a1d85157a0e65e98070f80f1bb561 100644 (file)
@@ -74,8 +74,8 @@ public class CrosscuttingMembers {
                this.shouldConcretizeIfNeeded = shouldConcretizeIfNeeded;
        }
 
-       private final Hashtable<String, Object> cflowFields = new Hashtable<>();
-       private final Hashtable<String, Object> cflowBelowFields = new Hashtable<>();
+       private final Map<String, Object> cflowFields = new Hashtable<>();
+       private final Map<String, Object> cflowBelowFields = new Hashtable<>();
 
        // public void addConcreteShadowMungers(Collection c) {
        // shadowMungers.addAll(c);
index 293a7766c769ac1a6853d0ff9d9bc8bbd2aa7e80..9a322d98f422b6b649a256c4b27f77dfe65a4455 100644 (file)
@@ -1425,13 +1425,13 @@ public abstract class ResolvedType extends UnresolvedType implements AnnotatedEl
         * ??? This method is O(N*M) where N = number of methods and M is number of inter-type declarations in my super
         */
        public List<ConcreteTypeMunger> getInterTypeMungersIncludingSupers() {
-               ArrayList<ConcreteTypeMunger> ret = new ArrayList<>();
+               List<ConcreteTypeMunger> ret = new ArrayList<>();
                collectInterTypeMungers(ret);
                return ret;
        }
 
        public List<ConcreteTypeMunger> getInterTypeParentMungersIncludingSupers() {
-               ArrayList<ConcreteTypeMunger> ret = new ArrayList<>();
+               List<ConcreteTypeMunger> ret = new ArrayList<>();
                collectInterTypeParentMungers(ret);
                return ret;
        }
index c6ebb14b39d2ca032d0c1dc1aea4d46152d8bd92..c055290ae37c9fc679e1d4d1a4a05dd56349ea3a 100644 (file)
@@ -2037,7 +2037,7 @@ public class AjcTask extends MatchingTask {
                // }
 
                List extractArguments() {
-                       ArrayList result = new ArrayList();
+                       List result = new ArrayList();
                        String[] cmds = command.getArguments();
                        if (!LangUtil.isEmpty(cmds)) {
                                result.addAll(Arrays.asList(cmds));
index 80e9ed4a09213ba8ea7372eefe165001e4cc547e..3a5f3b95475a08fb6530f6490af7e1d9d59c33cc 100644 (file)
@@ -20,6 +20,7 @@ import org.aspectj.util.FileUtil;
 import java.io.File;
 import java.util.ArrayList;
 import java.util.Iterator;
+import java.util.List;
 
 import junit.framework.TestCase;
 
@@ -28,7 +29,7 @@ import junit.framework.TestCase;
  */
 public class Ajc11CompilerAdapterTest extends TestCase {
     public static boolean LOGGING = false;
-    ArrayList tempFiles = new ArrayList();
+    List tempFiles = new ArrayList();
     
     public Ajc11CompilerAdapterTest(String name) {
         super(name);
index d42bcd9941aec90f2a8b11306d52a25fd5ff2d48..8e6e98db402d15c9506626f82d64679ec0ad09be 100644 (file)
@@ -720,7 +720,7 @@ public class Tester {
     private static String[] diffIgnoreDups(Collection<String> set, String[] expected, String msg,
         boolean ignoreDups) {
         ArrayList<String> result = new ArrayList<>();
-        ArrayList<String> actual = new ArrayList<>(set);
+        List<String> actual = new ArrayList<>(set);
         BitSet hits = new BitSet();
         for (int i = 0; i < expected.length; i++) {
             if (!actual.remove(expected[i])) {
index 5533739a386d4b3b85a8d2d2760feb73a9309cfe..e757dfc4bb42402b58b6e255609a5673c302da6f 100644 (file)
@@ -166,8 +166,8 @@ public class TesterTest extends TestCase {
      * @author isberg
      */
     public static class MyTestReporter implements IMessageHandler {
-        public ArrayList<IMessage> failures = new ArrayList<>();
-        public ArrayList<IMessage> passes = new ArrayList<>();
+        public List<IMessage> failures = new ArrayList<>();
+        public List<IMessage> passes = new ArrayList<>();
 
         public void clear() {
             failures.clear();
index 42f4c4451dbb985fd958fdc3a6050b4d44c58900..751a035fa45d55eed1163b4be650472ac50b98c5 100644 (file)
@@ -176,7 +176,7 @@ public class Harness {
        /** just don't say anything! */
        protected boolean silentHarness;
 
-       private HashMap<String,Feature> features;
+       private Map<String,Feature> features;
 
        /** if true, do not delete temporary files. */
        private boolean keepTemp;
@@ -212,7 +212,7 @@ public class Harness {
                LangUtil.throwIaxIfFalse(!LangUtil.isEmpty(args), "empty args");
                // read arguments
                final ArrayList<String> globals = new ArrayList<>();
-               final ArrayList<String> files = new ArrayList<>();
+               final List<String> files = new ArrayList<>();
                final LinkedList<String> argList = new LinkedList<>(Arrays.asList(args));
                for (int i = 0; i < argList.size(); i++) {
                        String arg = argList.get(i);
@@ -604,7 +604,7 @@ public class Harness {
        /** print known aliases at the end of the syntax message */
        protected void printAliases(PrintStream out) {
                LangUtil.throwIaxIfNull(out, "out");
-               Properties props = getOptionAliases();
+               Map props = getOptionAliases();
                if (null == props) {
                        return;
                }
@@ -699,7 +699,7 @@ class FeatureHarness extends Harness {
                                                                                                        "-junit" + OPTION_DELIM + "-ajctestSkipKeywords=knownLimitation,purejava"
                        };
        static {
-               Properties optionAliases = Harness.getOptionAliases();
+               Map optionAliases = Harness.getOptionAliases();
                if (null != optionAliases) {
                        for (int i = 1; i < ALIASES.length; i += 2) {
                                optionAliases.put(ALIASES[i-1], ALIASES[i]);
@@ -941,7 +941,7 @@ class TestMaker  {
        }
 
        public Harness.RunResult run(AjcTest.Suite.Spec spec) {
-               ArrayList<IRunSpec> kids = spec.getChildren();
+               Iterable<IRunSpec> kids = spec.getChildren();
                for (IRunSpec iRunSpec : kids) {
                        makeTest( (AjcTest.Spec) iRunSpec);
                }
index cf615f6d1d33ce30c1b5e0911666ac976dce4d49..ed4f560c0776332b82da854bcb5386253c2c59b5 100644 (file)
@@ -83,7 +83,7 @@ abstract public class AbstractRunSpec implements IRunSpec {
        private boolean skipAll;
 
        protected String xmlElementName; // nonfinal only for clone()
-       protected final ArrayList<String> keywords;
+       protected final List<String> keywords;
        protected final IMessageHolder /* IMessage */messages;
        protected final ArrayList<String> options;
        protected final ArrayList<String> paths;
@@ -197,7 +197,7 @@ abstract public class AbstractRunSpec implements IRunSpec {
                }
        }
 
-       public ArrayList<String> getKeywordsList() {
+       public List<String> getKeywordsList() {
                return makeList(keywords);
        }
 
@@ -257,7 +257,7 @@ abstract public class AbstractRunSpec implements IRunSpec {
 
        // --------------- (String) paths
        /** @return ArrayList of String paths */
-       public ArrayList<String> getPathsList() {
+       public List<String> getPathsList() {
                return makeList(paths);
        }
 
index 52a3e2266e83f3c9e31c6485070fb277eb79a9dd..2ff1733628ade0eb9c231e71df4d90fb075047a3 100644 (file)
@@ -250,7 +250,7 @@ public class AjcSpecTest extends TestCase {
                sameDirChangesList(lhs.dirChanges, rhs.dirChanges, a);
        }
 
-       public static void sameDirChangesList(ArrayList<DirChanges.Spec> lhs, ArrayList<DirChanges.Spec> rhs, Assert a) {
+       public static void sameDirChangesList(List<DirChanges.Spec> lhs, List<DirChanges.Spec> rhs, Assert a) {
                if ((null == lhs) && (null == rhs)) {
                        return;
                }
index 7bbc50adf4fce76dc4cba785adf7282ae6a77688..5ca41509f42753bbc7f0a320982b77cc2f970422 100644 (file)
@@ -161,7 +161,7 @@ public class AjcTest extends RunSpecIterator {
          * @return the unmodifiable List of titles (maybe empty, never null)
          */
         private static List<String> parseTitlesList(String titlesList) {
-            ArrayList<String> result = new ArrayList<>();
+            List<String> result = new ArrayList<>();
             String last = null;
             StringTokenizer st = new StringTokenizer(titlesList, ",");
             while (st.hasMoreTokens()) {
@@ -206,7 +206,7 @@ public class AjcTest extends RunSpecIterator {
          * @return the unmodifiable List of titles (maybe empty, never null)
          */
         private static List<String> readTitlesFile(File titlesFile, boolean fail) {
-            ArrayList<String> result = new ArrayList<>();
+            List<String> result = new ArrayList<>();
             Reader reader = null;
             try {
                 reader = new FileReader(titlesFile);
index 7a5bff3df1ea4b79949dcc103a43131a2446d5b3..25ca084bff03e0719eaf8905de68ea8b4a7981af 100644 (file)
@@ -729,7 +729,7 @@ public class CompilerRun implements IAjcRun {
         private static String updateBootclasspathForSourceVersion(
             String sourceVersion,
             String compilerName,
-            ArrayList toAdd) {
+            List toAdd) {
             if (null == sourceVersion) {
                 return null;
             }
@@ -1497,7 +1497,7 @@ public class CompilerRun implements IAjcRun {
             String seek;
 
             /** if setup completed, this has the combined global/local options */
-            ArrayList commandOptions;
+            List commandOptions;
 
             public Object clone() {
                 TestSetup testSetup = new TestSetup();
index e3e60e0280f7eff4277ddad87a38d714e1301d26..18b05982c8196431a5cf1446852089dec683040e 100644 (file)
@@ -469,19 +469,19 @@ public class DirChanges {
         boolean fastFail;
         
         /** relative paths (String) of expected files added */
-        final ArrayList<String> added;
+        final List<String> added;
 
         /** relative paths (String) of expected files removed/deleted */
-        final ArrayList<String> removed;
+        final List<String> removed;
 
         /** relative paths (String) of expected files updated/changed */
-        final ArrayList<String> updated;
+        final List<String> updated;
 
         /** relative paths (String) of expected files NOT 
          * added, removed, or changed 
          * XXX unchanged unimplemented
          */
-        final ArrayList<String> unchanged;
+        final List<String> unchanged;
     
         public Spec() {
             added = new ArrayList<>();
index 28db821cb4b4c0ba2db635c0b24309021a0682d4..696d491a563b734edd54ed3b94b5d70d05eae9af 100644 (file)
@@ -150,7 +150,7 @@ public class FlatSuiteReader implements SFileReader.Maker {
                        throw new AbortException("expected sources at " + reader);
                }
 
-               ArrayList<Message> exp = new ArrayList<>();
+               List<Message> exp = new ArrayList<>();
                // !compile || noerrors || className {runOption..}
                String first = words[0];
                if ("!compile".equals(first)) {
@@ -290,7 +290,7 @@ public class FlatSuiteReader implements SFileReader.Maker {
         result.description = input;
     
         ArrayList<String> newOptions = new ArrayList<>();
-        ArrayList<String> optionsCopy = result.getOptionsList();
+        Iterable<String> optionsCopy = result.getOptionsList();
         for (String option: optionsCopy) {
                        if (option.startsWith("-")) {
                 newOptions.add("!" + option.substring(1));
@@ -325,7 +325,7 @@ public class FlatSuiteReader implements SFileReader.Maker {
         */
        private List<Message> makeMessages(// XXX weak - also support expected exceptions, etc.
        Kind kind, String[] words, int start, File lastFile) {
-               ArrayList<Message> result = new ArrayList<>();
+               List<Message> result = new ArrayList<>();
                for (int i = start; i < words.length; i++) {
                        ISourceLocation sl =
                                BridgeUtil.makeSourceLocation(words[i], lastFile);
index ad62c97350973678099de80637a56dcf24ba67c4..22e52ec61aae286a636bb1ea1a64260ac6a184f4 100644 (file)
@@ -71,7 +71,7 @@ public class Globals {
                 && FileUtil.canReadFile(F_aspectjrt_jar)
                 && FileUtil.canReadFile(J2SE13_RTJAR)
                 && FileUtil.canReadFile(J2SE14_RTJAR));
-        HashMap map = new HashMap();
+        Map map = new HashMap();
         map.put("1.2", "java.lang.ref.Reference");
         map.put("1.3", "java.lang.reflect.Proxy");
         map.put("1.4", "java.nio.Buffer");
index a0c48878eb0da7f7142f4545554e94431f7cc99a..e3bbb9209b4e0822f739d033cdc66f074c6518b5 100644 (file)
@@ -409,7 +409,7 @@ public class IncCompilerRun implements IAjcRun {
             addItems(classesRemoved, items);
         }
         
-        private void addItems(ArrayList list, String items) {
+        private void addItems(List list, String items) {
             if (null != items) {
                 String[] classes = XMLWriter.unflattenList(items);
                 if (!LangUtil.isEmpty(classes)) {
index a0b4f5e9698eaf5c0ae37bc2b67fc52df3739e7a..1bda6adda8a5f1b119d39d3531f35000b13b1ddd 100644 (file)
 package org.aspectj.testing.harness.bridge;
 
 import java.io.File;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collection;
-import java.util.Iterator;
-import java.util.ListIterator;
-import java.util.Stack;
+import java.util.*;
 
 import org.aspectj.bridge.AbortException;
 import org.aspectj.bridge.IMessage;
@@ -51,10 +46,10 @@ public class Validator {
        private final Stack<IMessageHandler> handlers;
 
        /** list of File registered for deletion on demand */
-       private final ArrayList<File> tempFiles; // deleteTempFiles requires ListIterator.remove()
+       private final List<File> tempFiles; // deleteTempFiles requires ListIterator.remove()
 
        /** list of Sandboxes registered for cleanup on demand  */
-       private final ArrayList<Sandbox> sandboxes;
+       private final List<Sandbox> sandboxes;
 
        /** if true, throw AbortException on failure */
        boolean abortOnFailure;
index 4eb8e7a740501fd2be32a274e678746687425ff3..ebdc227d0f6e3179ac1a6211d1a409b3efdc3f01 100644 (file)
@@ -16,6 +16,7 @@ package org.aspectj.testing.run;
 
 import java.util.ArrayList;
 import java.util.Iterator;
+import java.util.List;
 
 /**
  * Aggregate listeners into one and run synchronously in order added.
@@ -23,7 +24,7 @@ import java.util.Iterator;
  */
 public class RunListeners extends RunListener implements IRunListener {
 
-    ArrayList listeners;
+    List listeners;
     public RunListeners() {
         listeners = new ArrayList();
     }
index 3871ee74b3755ccb41cfba34064bfb107060591f..b20a5650a01c411b487237165ad0e061194b3a3f 100644 (file)
@@ -14,6 +14,7 @@ package org.aspectj.testing.taskdefs;
 
 import java.io.File;
 import java.util.ArrayList;
+import java.util.List;
 
 import org.aspectj.bridge.IMessageHolder;
 import org.aspectj.bridge.MessageHandler;
@@ -33,7 +34,7 @@ import junit.framework.TestCase;
 public class AjcTaskCompileCommandTest extends TestCase {
     static boolean loggedWarning = false;
     static boolean runAllTests = true;
-    static ArrayList<File> tempFiles = new ArrayList<>();
+    static List<File> tempFiles = new ArrayList<>();
     
     private static File getClassesDir() {
         File tempDir = FileUtil.getTempDir("AjcTaskCompileCommandTest-classes");
@@ -41,7 +42,7 @@ public class AjcTaskCompileCommandTest extends TestCase {
         return tempDir;
     }
 
-    private static void addCommonArgs(ArrayList<String> list) {
+    private static void addCommonArgs(List<String> list) {
         list.add("-d");
         list.add(getClassesDir().getAbsolutePath());
         list.add("-classpath");
index e95ab3184b6fdcc830fd52eaca12ef8c6584eb6d..43739b7521328bb2258e2b511dccd7f66cd3746d 100644 (file)
@@ -148,8 +148,8 @@ public class Diffs {
                IMessage[] actual,
                IMessage.Kind[] ignoreExpectedKinds,
                IMessage.Kind[] ignoreActualKinds) {
-               ArrayList exp = getExcept(expected, ignoreExpectedKinds);
-               ArrayList act = getExcept(actual, ignoreActualKinds);
+               List exp = getExcept(expected, ignoreExpectedKinds);
+               List act = getExcept(actual, ignoreActualKinds);
 
                ArrayList missing = new ArrayList();
                List unexpected = new ArrayList();
@@ -161,8 +161,8 @@ public class Diffs {
                } else {
                        ListIterator expectedIterator = exp.listIterator();
                        int lastLine = Integer.MIN_VALUE + 1;
-                       ArrayList expectedFound = new ArrayList();
-                       ArrayList expectedForLine = new ArrayList();
+                       List expectedFound = new ArrayList();
+                       List expectedForLine = new ArrayList();
                        for (ListIterator iter = act.listIterator(); iter.hasNext();) {
                                IMessage actualMessage = (IMessage) iter.next();
                                int actualLine = getLine(actualMessage);
index 71dfb7f2e5604b0ba1ec3948f86485cd00367407..4f9f56424aac6e13855a776dd645c45db3c85ca5 100644 (file)
@@ -25,12 +25,7 @@ import java.io.StringBufferInputStream;
 import java.io.StringWriter;
 import java.net.MalformedURLException;
 import java.net.URL;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collection;
-import java.util.Enumeration;
-import java.util.Iterator;
-import java.util.Vector;
+import java.util.*;
 import java.util.jar.Attributes;
 import java.util.jar.JarOutputStream;
 import java.util.jar.Manifest;
@@ -131,7 +126,7 @@ public class FileUtil {
         final boolean checkExpected = !LangUtil.isEmpty(expectedPaths);
         
         // normalize sources to ignore
-        final ArrayList expected = (!checkExpected ? null : new ArrayList());
+        final List expected = (!checkExpected ? null : new ArrayList());
         if (checkExpected) {
                        for (String srcPath : expectedPaths) {
                                if (!LangUtil.isEmpty(srcPath)) {
@@ -186,7 +181,7 @@ public class FileUtil {
                 return false;
             }
         };
-               ArrayList unexp = new ArrayList(Arrays.asList(dir.listFiles(touchedCollector)));
+               List unexp = new ArrayList(Arrays.asList(dir.listFiles(touchedCollector)));
         
         // report any unexpected changes
         return Diffs.makeDiffs(label, expected, unexp, String.CASE_INSENSITIVE_ORDER);
index ad9331dd359819bae8bcf37d1bbd08d17d18be08..b4acc5ce3c6ffa2dd1bdc296a0cdb3236414b798 100644 (file)
@@ -402,7 +402,7 @@ public class LangUtil {
      * @see Util#combine(Object[], Object[])
      */
     public static String[] combine(String[] one, String[] two) {
-               ArrayList twoList = new ArrayList(org.aspectj.util.LangUtil.arrayAsList(two));
+               List twoList = new ArrayList(org.aspectj.util.LangUtil.arrayAsList(two));
         ArrayList result = new ArrayList();
         if (null != one) {
                        for (String s : one) {
@@ -938,10 +938,10 @@ public class LangUtil {
             return;
         }
 
-               ArrayList expected = new ArrayList(expectedListIn);
+               List expected = new ArrayList(expectedListIn);
         expected.sort(comparator);
 
-               ArrayList actual = new ArrayList(actualListIn);
+               List actual = new ArrayList(actualListIn);
         actual.sort(comparator);
         Iterator actualIter = actual.iterator();        
         Object act = null;
index 6eede56a6d352fa70f532f74529ee6a8169f0b86..853e51f50eba77962674e0ffe8ca22676f7df93b 100644 (file)
@@ -67,7 +67,7 @@ public class LangUtilTest extends TestCase {
 
        void checkUnflatten(FTest test) {
         String[] exp = test.unflattened;
-        ArrayList result = LangUtil.unflatten(test.toUnflatten, test.spec);
+        List result = LangUtil.unflatten(test.toUnflatten, test.spec);
         String label = test + " -> " + result;
         assertNotNull(label, result);
         
@@ -96,7 +96,7 @@ public class LangUtilTest extends TestCase {
     
 
        public void testArrayList() {
-        ArrayList l = new ArrayList();
+        List l = new ArrayList();
         l.add(null);
         l.add(null);
         assertTrue(null == l.get(0));
@@ -269,15 +269,15 @@ public class LangUtilTest extends TestCase {
         if (unmodifiable) {
             return Collections.unmodifiableList(Arrays.asList(ra));
         } else {
-                       ArrayList list = new ArrayList(Arrays.asList(ra));
+                       List list = new ArrayList(Arrays.asList(ra));
             return list;
         }
     }
     
     /** check both hard and soft - assuming list contain String */
     void checkDiff(List expected, List actual, List missing, List extra) {
-        ArrayList extraOut = new ArrayList();
-        ArrayList missingOut = new ArrayList();
+        List extraOut = new ArrayList();
+        List missingOut = new ArrayList();
         LangUtil.makeDiffs(expected, actual, missingOut, extraOut);
         checkSame(missing, missingOut);
         checkSame(extra, extraOut);
@@ -301,8 +301,8 @@ public class LangUtilTest extends TestCase {
 
     /** check only soft - assuming list contain String */
     void checkDiffSoft(List expected, List actual, List missing, List extra) {
-        ArrayList extraOut = new ArrayList();
-        ArrayList missingOut = new ArrayList();
+        List extraOut = new ArrayList();
+        List missingOut = new ArrayList();
         LangUtil.makeSoftDiffs(expected, actual, missingOut, extraOut,
                             String.CASE_INSENSITIVE_ORDER);
         checkSameSoft(missing, missingOut);
@@ -316,8 +316,8 @@ public class LangUtilTest extends TestCase {
         String label = one + "?=" + two;
         assertTrue(label, (null == one) == (null == two));
         if (null != one) {
-                       ArrayList aone = new ArrayList(one);
-            ArrayList atwo = new ArrayList();
+                       List aone = new ArrayList(one);
+            List atwo = new ArrayList();
             aone.addAll(two);
             Collections.sort(aone);
             Collections.sort(atwo);
index a576c4ae48e527f16e299fc2d49649012150a362..17dff157ed962e715655829047893aa7813fff82 100644 (file)
@@ -171,10 +171,10 @@ public class LinkCheck {
     
     private final Messages messages;
     private final HTMLEditorKit.Parser parser; // XXX untested - stateful
-    private final ArrayList<Link> linksToCheck;
-    private final ArrayList<String> checkedUrls;  // String (URL.toString)
-    private final ArrayList<String> validRefs;  // String (URL.toString)
-    private final ArrayList<String> refsToCheck;  // String (URL.toString)
+    private final List<Link> linksToCheck;
+    private final List<String> checkedUrls;  // String (URL.toString)
+    private final List<String> validRefs;  // String (URL.toString)
+    private final List<String> refsToCheck;  // String (URL.toString)
     
     private final Link.Check checkExists;
     private final Link.Check checkContents;
@@ -221,7 +221,7 @@ public class LinkCheck {
     }
 
     public synchronized void run() {
-        ArrayList<Link> list = new ArrayList<>();
+        List<Link> list = new ArrayList<>();
         while (0 < linksToCheck.size()) {
             messages.checkingLinks(linksToCheck.size());
             list.clear();
index abfff1ee6d6c9bc302b21783fba7ba58c1c2e456..9dc379d848d17f1ca3d744095a1aba702450bacf 100644 (file)
@@ -71,7 +71,7 @@ public class MessageUtilTest extends TestCase {
 
     List getSampleMessageTexts() {
         if (null == messageTexts) {
-                       ArrayList result = new ArrayList(Arrays.asList(new String[]
+                       List result = new ArrayList(Arrays.asList(new String[]
                                        {"one", "two", "now is the time for all good men..."}));
             messageTexts = result;
         }
@@ -80,7 +80,7 @@ public class MessageUtilTest extends TestCase {
     
     List getSampleExceptions() {
         if (null == exceptions) {
-            ArrayList result = new ArrayList();
+            List result = new ArrayList();
             int i = 1;
             result.add(new Error("Error " + i++));
             result.add(new RuntimeException("RuntimeException " + i++));
@@ -92,7 +92,7 @@ public class MessageUtilTest extends TestCase {
 
     List getSampleLocations() {
         if (null == locations) {
-            ArrayList result = new ArrayList();
+            List result = new ArrayList();
             File file = new File("testsrc/org/aspectj/testing/util/MessageUtilTest.java");
             result.add(new SourceLocation(file, 1, 2, 1));
             result.add(new SourceLocation(file, 100, 100, 0));
index 8a753dfc7a51ebd6bdbe10a9c28050758eba101b..559df2a0f7d98ca357a248b6e1f9d5fdea1abd89 100644 (file)
@@ -41,7 +41,7 @@ public class TestClassLoader extends URLClassLoader {
         super(urls);
         this.urlsForDebugString = urls;
         LangUtil.throwIaxIfComponentsBad(dirs, "dirs", null);
-        ArrayList dcopy = new ArrayList();
+        List dcopy = new ArrayList();
         
         if (!LangUtil.isEmpty(dirs)) {
             dcopy.addAll(Arrays.asList(dirs));
index 11dd155874423a18d75c7b552e2709fd011d4a01..09391bd6d9bf4829c94ecbc6dad375e558658147 100644 (file)
@@ -83,8 +83,8 @@ public class TestDiffs { // XXX pretty dumb implementation
             Diffs tests = Diffs.makeDiffs("tests", exp, act, TestResult.BY_NAME);
             // remove missing/unexpected (removed, added) tests from results
             // otherwise, unexpected-[pass|fail] look like [fixes|broken]
-            ArrayList expResults = trimByName(exp, tests.missing);
-            ArrayList actResults = trimByName(act, tests.unexpected);
+            List expResults = trimByName(exp, tests.missing);
+            List actResults = trimByName(act, tests.unexpected);
             
             Diffs results = Diffs.makeDiffs("results", expResults, actResults, TestResult.BY_PASSNAME);
 
@@ -159,7 +159,7 @@ public class TestDiffs { // XXX pretty dumb implementation
 
     
     /** split input List by whether the TestResult element passed or failed */
-    private static void split(List input, ArrayList pass, ArrayList fail) {
+    private static void split(List input, List pass, List fail) {
                for (Object o : input) {
                        TestResult result = (TestResult) o;
                        if (result.pass) {
index 17fa8234da4ce4a9a4220e741adb6c8c2e0a5957..a0c5e115ff4916b1377497022fe875ced9d5a655 100644 (file)
@@ -18,6 +18,7 @@ import java.io.FileWriter;
 import java.io.IOException;
 import java.io.PrintWriter;
 import java.util.ArrayList;
+import java.util.List;
 import java.util.ListIterator;
 
 import org.aspectj.util.FileUtil;
@@ -67,7 +68,7 @@ public class TestDiffsTest extends TestCase {
         }
     }
 
-    ArrayList tempFiles;
+    List tempFiles;
        /**
         * Constructor for FileUtilTest.
         * @param arg0
index b9c6d2cab477f6fcb3d9c7731c13f24155a2e499..5d3ec4ba6f2ef6af5492750323223fd04ad6f2fb 100644 (file)
@@ -365,7 +365,7 @@ public class Option implements Comparable {
         private final Map familyNameToFamily = new TreeMap();
 
         /** enforce uniqueness of options */
-        private final ArrayList names = new ArrayList();
+        private final List names = new ArrayList();
 
         public Factory(String factoryName) {
             this.factoryName = factoryName;
index 92e84c106ab66bde28376fea8a7a2345971446ac..03b3a6d51249c4e99b9b63cbd9402deda27cce0c 100644 (file)
@@ -14,6 +14,7 @@ package org.aspectj.testing.util.options;
 
 import java.util.ArrayList;
 import java.util.Iterator;
+import java.util.List;
 
 import org.aspectj.util.LangUtil;
 
@@ -59,7 +60,7 @@ public class Options {
         return sb.toString();
     }
 
-    private final ArrayList options = new ArrayList();
+    private final List options = new ArrayList();
     private final boolean stopAtFirstMatch;
     private boolean frozen = !FROZEN;
 
index d6ebc774e8f3d8d3793e73c7739a98c80f744832..b60e56470c91d090d820a42605dc0bb7fa29b590 100644 (file)
@@ -21,6 +21,7 @@ import java.io.File;
 import java.io.IOException;
 import java.util.ArrayList;
 import java.util.Iterator;
+import java.util.List;
 
 import org.aspectj.testing.harness.bridge.AjcSpecTest;
 import org.aspectj.testing.harness.bridge.AjcTest;
@@ -35,7 +36,7 @@ import junit.framework.TestCase;
  */
 public class AjcSpecXmlReaderTest extends TestCase {
 
-       ArrayList<File> tempFiles = new ArrayList<>();
+       List<File> tempFiles = new ArrayList<>();
        /**
         * Constructor for AjcSpecXmlReaderTest.
         * @param name
@@ -111,7 +112,7 @@ public class AjcSpecXmlReaderTest extends TestCase {
         String xml2Path = path + ".tmp.xml";
 
         final File file1 = new File(xmlPath);
-        final ArrayList<File> toDelete = new ArrayList<>();
+        final List<File> toDelete = new ArrayList<>();
         final AjcSpecXmlReader writer = AjcSpecXmlReader.getReader();
 
         assertTrue("" + file1, file1.canRead());
@@ -154,7 +155,7 @@ public class AjcSpecXmlReaderTest extends TestCase {
         AjcSpecXmlReader writer = AjcSpecXmlReader.getReader();
         File file0 = new File(txtPath);
         File file1 = new File(xmlPath);
-        ArrayList<File> toDelete = new ArrayList<>();
+        List<File> toDelete = new ArrayList<>();
         AjcTest.Suite.Spec suite0 = null;
         if (file0.canRead()) {
             System.out.println("reading " + file0);
index 342484a76000addc23f59e9155de558f74150272..3201d4288adbc0166dff22b93fe3b41337953a94 100644 (file)
@@ -16,6 +16,7 @@ package org.aspectj.testing.xml;
 import java.io.File;
 import java.util.ArrayList;
 import java.util.Iterator;
+import java.util.List;
 
 import org.aspectj.bridge.IMessage;
 import org.aspectj.util.LangUtil;
@@ -27,7 +28,7 @@ import junit.framework.TestCase;
  */
 public class MessageListXmlReaderTest extends TestCase {
 
-       ArrayList tempFiles = new ArrayList();
+       List tempFiles = new ArrayList();
        public MessageListXmlReaderTest(String name) {
                super(name);
        }
index 969c8ecd8d0fc92d6d7ee3882c067ef88ad474d3..8199e3826e4e74dc52255ce04488a99e06ae3d1d 100644 (file)
@@ -41,7 +41,7 @@ public class SoftMessage implements IMessage {
        private String details;
        private int id;
        private int sourceStart,sourceEnd;
-       private final ArrayList extraSourceLocations = new ArrayList();
+       private final List extraSourceLocations = new ArrayList();
 
        //private ISourceLocation pseudoSourceLocation;  // set directly
        // collapse enclosed source location for shorter, property-based xml
index 5e6708a2e51022dca8a82eac86070fef30cab36a..086c8ce95c0c761d1aed87414254ddb5cb2192f7 100644 (file)
@@ -14,6 +14,7 @@ import java.io.File;
 import java.util.ArrayList;
 import java.util.Hashtable;
 import java.util.List;
+import java.util.Map;
 
 import org.aspectj.util.FileUtil;
 
@@ -42,7 +43,7 @@ public class AnnotationProcessingTests extends AbstractMultiProjectIncrementalAj
                configureProcessorOptions("ProcessorConsumer1","DemoProcessor");
                configureNonStandardCompileOptions("ProcessorConsumer1", "-showWeaveInfo");
 
-               Hashtable<String, String> javaOptions = new Hashtable<>();
+               Map<String, String> javaOptions = new Hashtable<>();
                javaOptions.put("org.eclipse.jdt.core.compiler.compliance", "1.6");
                javaOptions.put("org.eclipse.jdt.core.compiler.codegen.targetPlatform", "1.6");
                javaOptions.put("org.eclipse.jdt.core.compiler.source", "1.6");
@@ -70,7 +71,7 @@ public class AnnotationProcessingTests extends AbstractMultiProjectIncrementalAj
                configureProcessorPath("ProcessorConsumer2", getCompilerForProjectWithName("ProcessorProject2").getCompilerConfiguration().getOutputLocationManager().getDefaultOutputLocation().toString()+File.pathSeparator+
                                new File(testdataSrcDir + File.separatorChar + "ProcessorProject2" + File.separatorChar + "base"+File.separatorChar+"src").toString());
                
-               Hashtable<String, String> javaOptions = new Hashtable<>();
+               Map<String, String> javaOptions = new Hashtable<>();
                javaOptions.put("org.eclipse.jdt.core.compiler.compliance", "1.6");
                javaOptions.put("org.eclipse.jdt.core.compiler.codegen.targetPlatform", "1.6");
                javaOptions.put("org.eclipse.jdt.core.compiler.source", "1.6");
@@ -108,7 +109,7 @@ public class AnnotationProcessingTests extends AbstractMultiProjectIncrementalAj
 
                configureNonStandardCompileOptions("ProcessorConsumer1", "-showWeaveInfo");
 
-               Hashtable<String, String> javaOptions = new Hashtable<>();
+               Map<String, String> javaOptions = new Hashtable<>();
                javaOptions.put("org.eclipse.jdt.core.compiler.compliance", "1.6");
                javaOptions.put("org.eclipse.jdt.core.compiler.codegen.targetPlatform", "1.6");
                javaOptions.put("org.eclipse.jdt.core.compiler.source", "1.6");
index 36be6773ed5c7be252e6d68431fffcbaaa0f7e07..b3ae1b6c100f8160baafd9a2e937ff26e05e5249 100644 (file)
@@ -107,7 +107,7 @@ public class MultiProjTestCompilerConfiguration implements ICompilerConfiguratio
                if (javaOptionsMap != null && !javaOptionsMap.isEmpty())
                        return javaOptionsMap;
 
-               Hashtable<String, String> ht = new Hashtable<>();
+               Map<String, String> ht = new Hashtable<>();
                ht.put("org.eclipse.jdt.core.compiler.compliance", "1.5");
                ht.put("org.eclipse.jdt.core.compiler.codegen.targetPlatform", "1.5");
                ht.put("org.eclipse.jdt.core.compiler.source", "1.5");
index 49663211fe92ab1c517a7d09e63d4ce784b99291..3c3b0c72d20a3bee25df39530a52b37d6e5b13e6 100644 (file)
@@ -808,7 +808,7 @@ public class MultiProjectIncrementalTests extends AbstractMultiProjectIncrementa
                String p = "pr266420";
                initialiseProject(p);
 
-               Hashtable<String,String> javaOptions = new Hashtable<>();
+               Map<String,String> javaOptions = new Hashtable<>();
                javaOptions.put("org.eclipse.jdt.core.compiler.compliance", "1.6");
                javaOptions.put("org.eclipse.jdt.core.compiler.codegen.targetPlatform", "1.6");
                javaOptions.put("org.eclipse.jdt.core.compiler.source", "1.6");
@@ -3614,7 +3614,7 @@ public class MultiProjectIncrementalTests extends AbstractMultiProjectIncrementa
        public void testPR164384_1() {
                initialiseProject("PR164384");
 
-               Hashtable<String, String> javaOptions = new Hashtable<>();
+               Map<String, String> javaOptions = new Hashtable<>();
                javaOptions.put("org.eclipse.jdt.core.compiler.compliance", "1.6");
                javaOptions.put("org.eclipse.jdt.core.compiler.codegen.targetPlatform", "1.6");
                javaOptions.put("org.eclipse.jdt.core.compiler.source", "1.6");
@@ -3646,7 +3646,7 @@ public class MultiProjectIncrementalTests extends AbstractMultiProjectIncrementa
        public void testPR164384_2() {
                initialiseProject("PR164384");
 
-               Hashtable<String, String> javaOptions = new Hashtable<>();
+               Map<String, String> javaOptions = new Hashtable<>();
                javaOptions.put("org.eclipse.jdt.core.compiler.compliance", "1.6");
                javaOptions.put("org.eclipse.jdt.core.compiler.codegen.targetPlatform", "1.5");
                javaOptions.put("org.eclipse.jdt.core.compiler.source", "1.5");
@@ -3676,7 +3676,7 @@ public class MultiProjectIncrementalTests extends AbstractMultiProjectIncrementa
        public void testPR164384_3() {
                initialiseProject("PR164384");
 
-               Hashtable<String, String> javaOptions = new Hashtable<>();
+               Map<String, String> javaOptions = new Hashtable<>();
                javaOptions.put("org.eclipse.jdt.core.compiler.compliance", "1.6");
                javaOptions.put("org.eclipse.jdt.core.compiler.codegen.targetPlatform", "1.6");
                javaOptions.put("org.eclipse.jdt.core.compiler.source", "1.5");
index a586b69f69999941ad0df378b8fcca8a2cfad1ab..363056491af14f186b8fca85adeeaa6c008e873b 100644 (file)
@@ -1294,7 +1294,7 @@ public class FileUtil {
         * @param List sink the List for String entries of the form {sourcePath}:line:column
         * @return String error if any, or add String entries to sink
         */
-       public static String lineSeek(String sought, String sourcePath, boolean listAll, ArrayList<String> sink) {
+       public static String lineSeek(String sought, String sourcePath, boolean listAll, List<String> sink) {
                if (LangUtil.isEmpty(sought) || LangUtil.isEmpty(sourcePath)) {
                        return "nothing sought";
                }
index e97438b0e07b9784d973affe2e6642a1222c189e..edfc48f70d4ad7ef87e75cf9bc860ee1d26e8f7d 100644 (file)
@@ -359,7 +359,7 @@ public class LangUtil {
                if (null == input) {
                        return Collections.emptyList();
                }
-               ArrayList<String> result = new ArrayList<>();
+               List<String> result = new ArrayList<>();
 
                if (LangUtil.isEmpty(delim) || (!input.contains(delim))) {
                        result.add(input.trim());
@@ -741,7 +741,7 @@ public class LangUtil {
                final int sinkLength = (null == sink ? 0 : sink.length);
 
                final int resultSize;
-               ArrayList<Object> result = null;
+               List<Object> result = null;
                if (0 == sourceLength) {
                        resultSize = 0;
                } else {
@@ -1000,7 +1000,7 @@ public class LangUtil {
                if ((null == array) || (1 > array.length)) {
                        return Collections.emptyList();
                }
-               ArrayList<T> list = new ArrayList<>(Arrays.asList(array));
+               List<T> list = new ArrayList<>(Arrays.asList(array));
                return list;
        }
 
index b4d09533f96ccd0f5563cbde04fa2325ea55729b..2cb522f90995c9739c19adb51b4388bc82a9b3bc 100644 (file)
@@ -40,7 +40,7 @@ public class UtilClassLoader extends URLClassLoader {
         super(urls);
         LangUtil.throwIaxIfNotAssignable(dirs, File.class, "dirs");
         this.urlsForDebugString = urls;
-        ArrayList<File> dcopy = new ArrayList<>();
+        List<File> dcopy = new ArrayList<>();
         
         if (!LangUtil.isEmpty(dirs)) {
             dcopy.addAll(Arrays.asList(dirs));
index f5466e2cff66dffdae09657c82500237e39b2827..29cd246e44ec679c2b633ae79a0b9d501a83d7c5 100644 (file)
@@ -62,7 +62,7 @@ public class FileUtilTest extends TestCase {
         * @throws AssertionFailedError if any names are not in dir
         */
        public static String[] dirContains(File dir, final String[] filenames) {
-               final ArrayList<String> sought = new ArrayList<>(LangUtil.arrayAsList(filenames));
+               final List<String> sought = new ArrayList<>(LangUtil.arrayAsList(filenames));
                FilenameFilter filter = new FilenameFilter() {
                        public boolean accept(File d, String name) {
                                return !sought.remove(name);
@@ -188,7 +188,7 @@ public class FileUtilTest extends TestCase {
        }
 
        /** List of File files or directories to delete when exiting */
-       final ArrayList<File> tempFiles;
+       final List<File> tempFiles;
 
        public FileUtilTest(String s) {
                super(s);
@@ -388,7 +388,7 @@ public class FileUtilTest extends TestCase {
        }
 
        public void testRandomFileString() {
-               ArrayList<String> results = new ArrayList<>();
+               List<String> results = new ArrayList<>();
                for (int i = 0; i < 1000; i++) {
                        String s = FileUtil.randomFileString();
                        if (results.contains(s)) {
@@ -462,7 +462,7 @@ public class FileUtilTest extends TestCase {
                List<String> sourceList = new ArrayList<>();
                sourceList.add(file.getPath());
 
-               final ArrayList<String> errors = new ArrayList<>();
+               final List<String> errors = new ArrayList<>();
                final PrintStream errorSink = new PrintStream(System.err, true) {
                        public void println(String error) {
                                errors.add(error);
@@ -502,7 +502,7 @@ public class FileUtilTest extends TestCase {
                        tempFiles.add(file);
                }
                // now test
-               final ArrayList<String> errors = new ArrayList<>();
+               final List<String> errors = new ArrayList<>();
                final PrintStream errorSink = new PrintStream(System.err, true) {
                        public void println(String error) {
                                errors.add(error);
index b80080ec1d54e42660df7abc129616aeeb0fb666..110d58c8853ca2f6ef6f8bd382d0538d4e364254 100644 (file)
@@ -946,7 +946,7 @@ public class BcelWeaver {
 
                        manifest = new Manifest();
 
-                       Attributes attributes = manifest.getMainAttributes();
+                       Map attributes = manifest.getMainAttributes();
                        attributes.put(Name.MANIFEST_VERSION, WEAVER_MANIFEST_VERSION);
                        attributes.put(CREATED_BY, WEAVER_CREATED_BY);
                }
index cd378dbbf695cff63448fe7ff1c3221c00c2afd6..76acc6e7f5bb0d50fb57ee72c5456fcfcf0ecff6 100644 (file)
@@ -352,12 +352,12 @@ public class ClassPathManager {
                                                FileSystem fs = null;
                                                try {
                                                        if (LangUtil.is19VMOrGreater()) {
-                                                               HashMap<String, String> env = new HashMap<>();
+                                                               Map<String, String> env = new HashMap<>();
                                                                env.put("java.home",  jdkHome);
                                                                fs = FileSystems.newFileSystem(JRT_URI, env);
                                                        } else {
                                                                URLClassLoader loader = new URLClassLoader(new URL[] { jrtPath });
-                                                               HashMap<String, ?> env = new HashMap<>();
+                                                               Map<String, ?> env = new HashMap<>();
                                                                fs = FileSystems.newFileSystem(JRT_URI, env, loader);
                                                        }
                                                        state = new JImageState(jrtFsPath, fs);
index 7e2541e6c7be200ff5725dfb6b1276a2c38c99ae..7bac02dad1d78f00453f66919ed912ac1686d0ad 100644 (file)
@@ -1359,7 +1359,7 @@ public final class LazyMethodGen implements Traceable {
        }
 
        private void addExceptionHandlers(MethodGen gen, Map<InstructionHandle, InstructionHandle> map,
-                       LinkedList<ExceptionRange> exnList) {
+                       Iterable<ExceptionRange> exnList) {
                // now add exception handlers
                for (ExceptionRange r : exnList) {
                        if (r.isEmpty()) {
@@ -1547,7 +1547,7 @@ public final class LazyMethodGen implements Traceable {
        // but I don't trust the only implementation, TreeSet, to do the right
        // thing.
 
-       /* private */static void insertHandler(ExceptionRange fresh, LinkedList<ExceptionRange> l) {
+       /* private */static void insertHandler(ExceptionRange fresh, List<ExceptionRange> l) {
                // Old implementation, simply: l.add(0,fresh);
                for (ListIterator<ExceptionRange> iter = l.listIterator(); iter.hasNext();) {
                        ExceptionRange r = iter.next();
index e49988a89cd9203e3e7ad6b89210dbe1b044d2c1..8d912a317f7e98f018915d0a1a2e7f839644aaea 100644 (file)
@@ -253,7 +253,7 @@ public class SimpleAOPParser {
        private static void traverse(SimpleAOPParser sap, LightXMLParser xml)
                        throws Exception {
                sap.startElement(xml.getName(), xml.getAttributes());
-               ArrayList childrens = xml.getChildrens();
+               Iterable childrens = xml.getChildrens();
                for (Object children : childrens) {
                        LightXMLParser child = (LightXMLParser) children;
                        traverse(sap, child);
index f9b020272ddcfefd2be6869af177fba0c91a1c37..fbe5052929ff5ec44c0978d8c6ac13860047fb22 100644 (file)
@@ -191,13 +191,13 @@ public class ReferenceTypeTestCase extends TestCase {
 
        public void testAssignable02() {
                List list = new ArrayList();
-               ArrayList arraylist = null;
+               List arraylist = null;
                List<String> listOfString = new ArrayList<>();
                List<?> listOfSomething = new ArrayList<Integer>();
-               ArrayList<?> arrayListOfSomething = null;
+               List<?> arrayListOfSomething = null;
                List<Number> listOfNumber = null;
                ArrayList<Number> arrayListOfNumber = null;
-               ArrayList<? extends Number> arrayListOfSomethingNumberish = null;
+               List<? extends Number> arrayListOfSomethingNumberish = null;
                List<? extends Number> listOfSomethingNumberish = new ArrayList<Integer>();
                List<? super Double> listOfSomethingSuperDouble = new ArrayList<Number>();
                List<Integer> listOfInteger = new ArrayList<>();
@@ -274,14 +274,14 @@ public class ReferenceTypeTestCase extends TestCase {
 
        public void testAssignable03_method_m2() {
                List list = new ArrayList();
-               ArrayList arraylist = null;
+               List arraylist = null;
                List<String> listOfString = new ArrayList<>();
                List<?> listOfSomething = new ArrayList<Integer>();
-               ArrayList<?> arrayListOfSomething = null;
+               List<?> arrayListOfSomething = null;
                List<Number> listOfNumber = null;
-               ArrayList<Number> arrayListOfNumber = null;
-               ArrayList<Integer> arrayListOfInteger = null;
-               ArrayList<? extends Number> arrayListOfSomethingNumberish = null;
+               List<Number> arrayListOfNumber = null;
+               List<Integer> arrayListOfInteger = null;
+               List<? extends Number> arrayListOfSomethingNumberish = null;
                List<? extends Number> listOfSomethingNumberish = new ArrayList<Integer>();
                List<? super Double> listOfSomethingSuperDouble = new ArrayList<Number>();
                List<Integer> listOfInteger = new ArrayList<>();
@@ -364,15 +364,15 @@ public class ReferenceTypeTestCase extends TestCase {
                ArrayList arraylist = null;
                List<String> listOfString = new ArrayList<>();
                List<?> listOfSomething = new ArrayList<Integer>();
-               ArrayList<?> arrayListOfSomething = null;
+               List<?> arrayListOfSomething = null;
                List<Number> listOfNumber = null;
-               ArrayList<Number> arrayListOfNumber = null;
+               List<Number> arrayListOfNumber = null;
                ArrayList<Integer> arrayListOfInteger = null;
-               ArrayList<? extends Number> arrayListOfSomethingNumberish = null;
+               List<? extends Number> arrayListOfSomethingNumberish = null;
                List<? extends Number> listOfSomethingNumberish = new ArrayList<Integer>();
                List<? super Double> listOfSomethingSuperDouble = new ArrayList<Number>();
                List<Integer> listOfInteger = new ArrayList<>();
-               ArrayList arrayList = null;
+               List arrayList = null;
                ArrayList<String> arrayListOfString;
                ArrayList<Integer> arraylistOfInteger;
                // interfaces too List<? extends A,B>
@@ -477,18 +477,18 @@ public class ReferenceTypeTestCase extends TestCase {
 
        public void testAssignable03_method_m4() {
                List list = new ArrayList();
-               ArrayList arraylist = null;
+               List arraylist = null;
                List<String> listOfString = new ArrayList<>();
                List<?> listOfSomething = new ArrayList<Integer>();
-               ArrayList<?> arrayListOfSomething = null;
+               List<?> arrayListOfSomething = null;
                List<Number> listOfNumber = null;
-               ArrayList<Number> arrayListOfNumber = null;
-               ArrayList<? extends Number> arrayListOfSomethingNumberish = null;
+               List<Number> arrayListOfNumber = null;
+               List<? extends Number> arrayListOfSomethingNumberish = null;
                List<? extends Number> listOfSomethingNumberish = new ArrayList<Integer>();
                List<? super Double> listOfSomethingSuperDouble = new ArrayList<Number>();
                List<Integer> listOfInteger = new ArrayList<>();
-               ArrayList<String> arrayListOfString;
-               ArrayList<Integer> arraylistOfInteger;
+               List<String> arrayListOfString;
+               List<Integer> arraylistOfInteger;
                // interfaces too List<? extends A,B>
 
                ReferenceType ajArrayListOfString = resolve("Pjava/util/ArrayList<Ljava/lang/String;>;");