Redundant Collection.addAll() call

Reports Collection.addAll() and Map.putAll() calls after instantiation of a collection using a constructor call without arguments. Such constructs can be replaced with a single call to a parametrized constructor which simplifies code. Also for some collections the replacement might be more performant.

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

1 files changed, 1 insertions, 2 deletions

diff --git a/testing-drivers/src/test/java/org/aspectj/testing/drivers/Harness.java b/testing-drivers/src/test/java/org/aspectj/testing/drivers/Harness.java
index 55295568a..42f4c4451 100644
--- a/testing-drivers/src/test/java/org/aspectj/testing/drivers/Harness.java
+++ b/testing-drivers/src/test/java/org/aspectj/testing/drivers/Harness.java
@@ -213,8 +213,7 @@ public class Harness {
 		// read arguments
 		final ArrayList<String> globals = new ArrayList<>();
 		final ArrayList<String> files = new ArrayList<>();
-		final LinkedList<String> argList = new LinkedList<>();
-		argList.addAll(Arrays.asList(args));
+		final LinkedList<String> argList = new LinkedList<>(Arrays.asList(args));
 		for (int i = 0; i < argList.size(); i++) {
 			String arg = argList.get(i);
 			List<String> aliases = aliasOptions(arg);