diff --git a/src/main/org/apache/tools/ant/AntClassLoader.java b/src/main/org/apache/tools/ant/AntClassLoader.java index 41a0cdeb5..d0ba4dab0 100644 --- a/src/main/org/apache/tools/ant/AntClassLoader.java +++ b/src/main/org/apache/tools/ant/AntClassLoader.java @@ -742,8 +742,7 @@ public class AntClassLoader extends ClassLoader implements SubBuildListener, Clo private InputStream loadResource(final String name) { // we need to search the components of the path to see if we can // find the class we want. - return Collections.list(pathComponents.elements()).stream() - .map(path -> getResourceStream(path, name)) + return pathComponents.stream().map(path -> getResourceStream(path, name)) .filter(Objects::nonNull).findFirst().orElse(null); } @@ -826,10 +825,8 @@ public class AntClassLoader extends ClassLoader implements SubBuildListener, Clo // TODO - shouldn't this always return false in isolated mode? - return Collections.list(loaderPackages.elements()).stream() - .noneMatch(resourceName::startsWith) - && (Collections.list(systemPackages.elements()).stream() - .anyMatch(resourceName::startsWith) || parentFirst); + return loaderPackages.stream().noneMatch(resourceName::startsWith) + && (systemPackages.stream().anyMatch(resourceName::startsWith) || parentFirst); } /** @@ -869,7 +866,7 @@ public class AntClassLoader extends ClassLoader implements SubBuildListener, Clo } else { // try and load from this loader if the parent either didn't find // it or wasn't consulted. - for (final File pathComponent : Collections.list(pathComponents.elements())) { + for (final File pathComponent : pathComponents) { url = getResourceURL(pathComponent, name); if (url != null) { log("Resource " + name + " loaded from ant loader", Project.MSG_DEBUG); @@ -1343,7 +1340,7 @@ public class AntClassLoader extends ClassLoader implements SubBuildListener, Clo // we need to search the components of the path to see if // we can find the class we want. final String classFilename = getClassFilename(name); - for (final File pathComponent : Collections.list(pathComponents.elements())) { + for (final File pathComponent : pathComponents) { InputStream stream = null; try { stream = getResourceStream(pathComponent, classFilename); diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/ejb/WebsphereDeploymentTool.java b/src/main/org/apache/tools/ant/taskdefs/optional/ejb/WebsphereDeploymentTool.java index 45bb25e29..eed937724 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/ejb/WebsphereDeploymentTool.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/ejb/WebsphereDeploymentTool.java @@ -737,7 +737,7 @@ public class WebsphereDeploymentTool extends GenericDeploymentTool { newJarStream.setLevel(0); // Copy files from old WebSphere jar - for (JarEntry je : Collections.list(wasEntries.elements())) { + for (JarEntry je : wasEntries.values()) { if (je.getCompressedSize() == -1 || je.getCompressedSize() == je.getSize()) { newJarStream.setLevel(0); diff --git a/src/main/org/apache/tools/ant/types/FilterSet.java b/src/main/org/apache/tools/ant/types/FilterSet.java index a3fefce53..7d3333d78 100644 --- a/src/main/org/apache/tools/ant/types/FilterSet.java +++ b/src/main/org/apache/tools/ant/types/FilterSet.java @@ -20,7 +20,6 @@ package org.apache.tools.ant.types; import java.io.File; import java.io.InputStream; import java.nio.file.Files; -import java.util.Collections; import java.util.Hashtable; import java.util.Map; import java.util.Properties; @@ -252,8 +251,7 @@ public class FilterSet extends DataType implements Cloneable { dieOnCircularReference(); if (filterHash == null) { filterHash = new Hashtable<>(getFilters().size()); - Collections.list(getFilters().elements()) - .forEach(filter -> filterHash.put(filter.getToken(), filter.getValue())); + getFilters().forEach(filter -> filterHash.put(filter.getToken(), filter.getValue())); } return filterHash; }