This is an automated email from the ASF dual-hosted git repository.

tommaso pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/opennlp.git


The following commit(s) were added to refs/heads/master by this push:
     new 30b6759  OPENNLP-1167 - Remove WV#toDouble/FloatArray (#295)
30b6759 is described below

commit 30b6759387b4402987f6f4a6100fdc78e8bf755f
Author: Tommaso Teofili <[email protected]>
AuthorDate: Fri Dec 15 10:49:07 2017 +0100

    OPENNLP-1167 - Remove WV#toDouble/FloatArray (#295)
---
 .../tools/util/wordvector/DoubleArrayVector.java       | 16 +++-------------
 .../tools/util/wordvector/FloatArrayVector.java        | 18 ++++--------------
 .../java/opennlp/tools/util/wordvector/WordVector.java |  3 ---
 .../opennlp/tools/util/wordvector/WordVectorTable.java |  1 +
 4 files changed, 8 insertions(+), 30 deletions(-)

diff --git 
a/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/DoubleArrayVector.java
 
b/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/DoubleArrayVector.java
index dcbee86..79d03a3 100644
--- 
a/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/DoubleArrayVector.java
+++ 
b/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/DoubleArrayVector.java
@@ -44,27 +44,17 @@ class DoubleArrayVector implements WordVector {
   }
 
   @Override
-  public float[] toFloatArray() {
+  public FloatBuffer toFloatBuffer() {
     float[] floatVector = new float[vector.length];
     for (int i = 0; i < floatVector.length ; i++) {
       floatVector[i] = (float) vector[i];
     }
-    return floatVector;
-  }
-
-  @Override
-  public double[] toDoubleArray() {
-    return toDoubleBuffer().array();
-  }
-
-  @Override
-  public FloatBuffer toFloatBuffer() {
-    return FloatBuffer.wrap(toFloatArray()).asReadOnlyBuffer();
+    return FloatBuffer.wrap(floatVector).asReadOnlyBuffer();
   }
 
   @Override
   public DoubleBuffer toDoubleBuffer() {
-    return DoubleBuffer.wrap(vector);
+    return DoubleBuffer.wrap(vector).asReadOnlyBuffer();
   }
 
   @Override
diff --git 
a/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/FloatArrayVector.java
 
b/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/FloatArrayVector.java
index c6be1f1..0ee7287 100644
--- 
a/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/FloatArrayVector.java
+++ 
b/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/FloatArrayVector.java
@@ -44,27 +44,17 @@ class FloatArrayVector implements WordVector {
   }
 
   @Override
-  public float[] toFloatArray() {
-    return toFloatBuffer().array();
+  public FloatBuffer toFloatBuffer() {
+    return FloatBuffer.wrap(vector).asReadOnlyBuffer();
   }
 
   @Override
-  public double[] toDoubleArray() {
+  public DoubleBuffer toDoubleBuffer() {
     double[] doubleVector = new double[vector.length];
     for (int i = 0; i < doubleVector.length ; i++) {
       doubleVector[i] = vector[i];
     }
-    return doubleVector;
-  }
-
-  @Override
-  public FloatBuffer toFloatBuffer() {
-    return FloatBuffer.wrap(vector).asReadOnlyBuffer();
-  }
-
-  @Override
-  public DoubleBuffer toDoubleBuffer() {
-    return DoubleBuffer.wrap(toDoubleArray());
+    return DoubleBuffer.wrap(doubleVector).asReadOnlyBuffer();
   }
 
   @Override
diff --git 
a/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/WordVector.java 
b/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/WordVector.java
index 94acb54..fea95b1 100644
--- a/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/WordVector.java
+++ b/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/WordVector.java
@@ -35,9 +35,6 @@ public interface WordVector {
   float getAsFloat(int index);
   double getAsDouble(int index);
 
-  float[] toFloatArray();
-  double[] toDoubleArray();
-
   FloatBuffer toFloatBuffer();
   DoubleBuffer toDoubleBuffer();
 
diff --git 
a/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/WordVectorTable.java
 
b/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/WordVectorTable.java
index bf00858..17c347a 100644
--- 
a/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/WordVectorTable.java
+++ 
b/opennlp-tools/src/main/java/opennlp/tools/util/wordvector/WordVectorTable.java
@@ -27,6 +27,7 @@ import opennlp.tools.util.java.Experimental;
  */
 @Experimental
 public interface WordVectorTable {
+
   WordVector get(CharSequence token);
 
   int size();

-- 
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].

Reply via email to