Skip to content

Commit 59cd07c

Browse files
committed
PrimitiveArrayConverters: Remove TODOs
1 parent 040e846 commit 59cd07c

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

scijava-ops-engine/src/main/java/org/scijava/ops/engine/matcher/convert/PrimitiveArrayConverters.java

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@ public class PrimitiveArrayConverters<N extends Number> implements
5151
@OpField(names = "engine.convert")
5252
public final Function<N[], ObjectArray<Number>> fromNumber = arr -> {
5353
var oa = new ObjectArray<>(Number.class, arr.length);
54-
// TODO: Why doesn't System.arraycopy work?
5554
for (int i = 0; i < arr.length; i++) {
5655
oa.set(i, arr[i]);
5756
}
@@ -65,7 +64,6 @@ public class PrimitiveArrayConverters<N extends Number> implements
6564
public final Function<byte[], ObjectArray<Number>> fromPrimitiveByte =
6665
arr -> {
6766
var oa = new ObjectArray<>(Number.class, arr.length);
68-
// TODO: Why doesn't System.arraycopy work?
6967
for (int i = 0; i < arr.length; i++) {
7068
oa.set(i, arr[i]);
7169
}
@@ -77,7 +75,6 @@ public class PrimitiveArrayConverters<N extends Number> implements
7775
public final Function<short[], ObjectArray<Number>> fromPrimitiveShort =
7876
arr -> {
7977
var oa = new ObjectArray<>(Number.class, arr.length);
80-
// TODO: Why doesn't System.arraycopy work?
8178
for (int i = 0; i < arr.length; i++) {
8279
oa.set(i, arr[i]);
8380
}
@@ -88,7 +85,6 @@ public class PrimitiveArrayConverters<N extends Number> implements
8885
@OpField(names = "engine.convert")
8986
public final Function<int[], ObjectArray<Number>> fromPrimitiveInt = arr -> {
9087
var oa = new ObjectArray<>(Number.class, arr.length);
91-
// TODO: Why doesn't System.arraycopy work?
9288
for (int i = 0; i < arr.length; i++) {
9389
oa.set(i, arr[i]);
9490
}
@@ -100,7 +96,6 @@ public class PrimitiveArrayConverters<N extends Number> implements
10096
public final Function<long[], ObjectArray<Number>> fromPrimitiveLong =
10197
arr -> {
10298
var oa = new ObjectArray<>(Number.class, arr.length);
103-
// TODO: Why doesn't System.arraycopy work?
10499
for (int i = 0; i < arr.length; i++) {
105100
oa.set(i, arr[i]);
106101
}
@@ -112,7 +107,6 @@ public class PrimitiveArrayConverters<N extends Number> implements
112107
public final Function<float[], ObjectArray<Number>> fromPrimitiveFloat =
113108
arr -> {
114109
var oa = new ObjectArray<>(Number.class, arr.length);
115-
// TODO: Why doesn't System.arraycopy work?
116110
for (int i = 0; i < arr.length; i++) {
117111
oa.set(i, arr[i]);
118112
}
@@ -124,7 +118,6 @@ public class PrimitiveArrayConverters<N extends Number> implements
124118
public final Function<double[], ObjectArray<Number>> fromPrimitiveDouble =
125119
arr -> {
126120
var oa = new ObjectArray<>(Number.class, arr.length);
127-
// TODO: Why doesn't System.arraycopy work?
128121
for (int i = 0; i < arr.length; i++) {
129122
oa.set(i, arr[i]);
130123
}

0 commit comments

Comments
 (0)