Use clone() instead of manual population of array copy

2 files changed, 2 insertions, 4 deletions

diff --git a/asm-commons/src/main/java/org/objectweb/asm/commons/Remapper.java b/asm-commons/src/main/java/org/objectweb/asm/commons/Remapper.java
index 4b1abf45..3b777638 100644
--- a/asm-commons/src/main/java/org/objectweb/asm/commons/Remapper.java
+++ b/asm-commons/src/main/java/org/objectweb/asm/commons/Remapper.java
@@ -110,8 +110,7 @@ public abstract class Remapper {
       String remappedInternalName = mapType(internalName);
       if (remappedInternalName != null) {
         if (remappedInternalNames == null) {
-          remappedInternalNames = new String[internalNames.length];
-          System.arraycopy(internalNames, 0, remappedInternalNames, 0, internalNames.length);
+          remappedInternalNames = internalNames.clone();
         }
         remappedInternalNames[i] = remappedInternalName;
       }
diff --git a/asm-commons/src/main/java/org/objectweb/asm/commons/SerialVersionUIDAdder.java b/asm-commons/src/main/java/org/objectweb/asm/commons/SerialVersionUIDAdder.java
index fbaf49d9..279f91ed 100644
--- a/asm-commons/src/main/java/org/objectweb/asm/commons/SerialVersionUIDAdder.java
+++ b/asm-commons/src/main/java/org/objectweb/asm/commons/SerialVersionUIDAdder.java
@@ -186,11 +186,10 @@ public class SerialVersionUIDAdder extends ClassVisitor {
     if (computeSvuid) {
       this.name = name;
       this.access = access;
-      this.interfaces = new String[interfaces.length];
+      this.interfaces = interfaces.clone();
       this.svuidFields = new ArrayList<>();
       this.svuidConstructors = new ArrayList<>();
       this.svuidMethods = new ArrayList<>();
-      System.arraycopy(interfaces, 0, this.interfaces, 0, interfaces.length);
     }
 
     super.visit(version, access, name, signature, superName, interfaces);