Defensive copy in addClasspathEntry so that it won't attempt to add a…

…n element to a read-only list
openrewrite · Mar 4, 2024 · 01b44eb · 01b44eb
1 parent 10a7498
commit 01b44eb
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/rewrite-java/src/main/java/org/openrewrite/java/JavaParser.java b/rewrite-java/src/main/java/org/openrewrite/java/JavaParser.java
@@ -371,6 +371,7 @@ B addClasspathEntry(Path entry) {
             if (this.classpath.isEmpty()) {
                 this.classpath = Collections.singletonList(entry);
             } else {
+                this.classpath = new ArrayList<>(classpath);
                 this.classpath.add(entry);
             }
             return (B) this;
@@ -382,7 +383,7 @@ public B classpath(String... artifactNames) {
             return (B) this;
         }
 
-        @SuppressWarnings("UnusedReturnValue")
+        @SuppressWarnings({"UnusedReturnValue", "unused"})
         public B classpathFromResources(ExecutionContext ctx, String... classpath) {
             this.artifactNames = Collections.emptyList();
             this.classpath = dependenciesFromResources(ctx, classpath);