# HG changeset patch # User attila # Date 1421680036 -3600 # Node ID 37b0c5bda589111b1c608785ec2b9837803bee94 # Parent 8327024a99559982b848e9c2191da9c0bf8838fd 8067880: Dead typed push methods in ArrayData Reviewed-by: hannesw, jlaskey diff -r 8327024a9955 -r 37b0c5bda589 nashorn/src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/arrays/ArrayData.java --- a/nashorn/src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/arrays/ArrayData.java Wed Jul 05 20:15:13 2017 +0200 +++ b/nashorn/src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/arrays/ArrayData.java Mon Jan 19 16:07:16 2015 +0100 @@ -26,6 +26,7 @@ package jdk.nashorn.internal.runtime.arrays; import static jdk.nashorn.internal.codegen.CompilerConstants.staticCall; + import java.lang.invoke.MethodHandle; import java.lang.invoke.MethodHandles; import java.lang.reflect.Array; @@ -761,39 +762,6 @@ } /** - * Push an array of items to the end of the array - * - * @param strict are we in strict mode - * @param item the item - * @return new array data (or same) - */ - public ArrayData push(final boolean strict, final double item) { - return push(strict, item); - } - - /** - * Push an array of items to the end of the array - * - * @param strict are we in strict mode - * @param item the item - * @return new array data (or same) - */ - public ArrayData push(final boolean strict, final long item) { - return push(strict, item); - } - - /** - * Push an array of items to the end of the array - * - * @param strict are we in strict mode - * @param item the item - * @return new array data (or same) - */ - public ArrayData push(final boolean strict, final int item) { - return push(strict, item); - } - - /** * Pop an element from the end of the array * * @return the popped element diff -r 8327024a9955 -r 37b0c5bda589 nashorn/src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/arrays/IntArrayData.java --- a/nashorn/src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/arrays/IntArrayData.java Wed Jul 05 20:15:13 2017 +0200 +++ b/nashorn/src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/arrays/IntArrayData.java Mon Jan 19 16:07:16 2015 +0100 @@ -26,6 +26,7 @@ package jdk.nashorn.internal.runtime.arrays; import static jdk.nashorn.internal.codegen.CompilerConstants.specialCall; + import java.lang.invoke.MethodHandle; import java.lang.invoke.MethodHandles; import java.util.Arrays; @@ -343,17 +344,6 @@ } @Override - public final ArrayData push(final boolean strict, final int item) { - final long len = length(); - final ArrayData newData = ensure(len); - if (newData == this) { - array[(int)len] = item; - return this; - } - return newData.set((int)len, item, strict); - } - - @Override public ArrayData fastSplice(final int start, final int removed, final int added) throws UnsupportedOperationException { final long oldLength = length(); final long newLength = oldLength - removed + added; diff -r 8327024a9955 -r 37b0c5bda589 nashorn/src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/arrays/LongArrayData.java --- a/nashorn/src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/arrays/LongArrayData.java Wed Jul 05 20:15:13 2017 +0200 +++ b/nashorn/src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/arrays/LongArrayData.java Mon Jan 19 16:07:16 2015 +0100 @@ -27,6 +27,7 @@ import static jdk.nashorn.internal.codegen.CompilerConstants.specialCall; import static jdk.nashorn.internal.lookup.Lookup.MH; + import java.lang.invoke.MethodHandle; import java.lang.invoke.MethodHandles; import java.util.Arrays; @@ -303,17 +304,6 @@ } @Override - public final ArrayData push(final boolean strict, final long item) { - final long len = length(); - final ArrayData newData = ensure(len); - if (newData == this) { - array[(int)len] = item; - return this; - } - return newData.set((int)len, item, strict); - } - - @Override public ArrayData fastSplice(final int start, final int removed, final int added) throws UnsupportedOperationException { final long oldLength = length(); final long newLength = oldLength - removed + added; diff -r 8327024a9955 -r 37b0c5bda589 nashorn/src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/arrays/NumberArrayData.java --- a/nashorn/src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/arrays/NumberArrayData.java Wed Jul 05 20:15:13 2017 +0200 +++ b/nashorn/src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/arrays/NumberArrayData.java Mon Jan 19 16:07:16 2015 +0100 @@ -28,6 +28,7 @@ import static jdk.nashorn.internal.codegen.CompilerConstants.specialCall; import static jdk.nashorn.internal.lookup.Lookup.MH; import static jdk.nashorn.internal.runtime.ScriptRuntime.UNDEFINED; + import java.lang.invoke.MethodHandle; import java.lang.invoke.MethodHandles; import java.util.Arrays; @@ -277,17 +278,6 @@ } @Override - public final ArrayData push(final boolean strict, final double item) { - final long len = length(); - final ArrayData newData = ensure(len); - if (newData == this) { - array[(int)len] = item; - return this; - } - return newData.set((int)len, item, strict); - } - - @Override public ArrayData fastSplice(final int start, final int removed, final int added) throws UnsupportedOperationException { final long oldLength = length(); final long newLength = oldLength - removed + added;