Fix ControlBinding::getValueArray with updated test.

Author: neilcsmith-net

src/org/freedesktop/gstreamer/ControlBinding.java

@@ -74,16 +74,16 @@ public Object getValue(long timestamp) {
      * @return false if the given array could not be filled
      */
     public boolean getValueArray(long timestamp, long interval, Object[] values) {
-        GValueAPI.GValueArray gValues = new GValueAPI.GValueArray(values.length);
+        GValueAPI.GValue[] gValues = new GValueAPI.GValue[values.length];
         boolean ok = GSTCONTROLBINDING_API.gst_control_binding_get_g_value_array(
                 handle.getPointer(),
                 timestamp,
                 interval,
-                gValues.n_values,
+                gValues.length,
                 gValues);
         if (ok) {
             for (int i = 0; i < values.length; i++) {
-                values[i] = gValues.getValue(i);
+                values[i] = gValues[i].getValue();
             }
         }
         return ok;

src/org/freedesktop/gstreamer/lowlevel/GstControlBindingAPI.java

@@ -44,7 +44,7 @@ boolean gst_control_binding_get_g_value_array(GstControlBindingPtr binding,
             long timestamp,
             long internal,
             int n_values,
-            GValueAPI.GValueArray values);
+            GValueAPI.GValue[] values);
 
     void gst_control_binding_set_disabled(GstControlBindingPtr binding,
             boolean disabled);

test/org/freedesktop/gstreamer/controller/InterpolationControlSourceTest.java

@@ -1,5 +1,6 @@
 package org.freedesktop.gstreamer.controller;
 
+import java.util.Arrays;
 import java.util.Collections;
 import java.util.List;
 import java.util.stream.Collectors;
@@ -16,6 +17,7 @@
 import org.junit.Before;
 import org.junit.BeforeClass;
 import org.junit.Test;
+
 import static org.junit.Assert.*;
 
 /**
@@ -115,13 +117,25 @@ public void testLinearInterpolationAbsolute() {
         controller.setFromList(timedValues);
 
         Element volume = ElementFactory.make("volume", "volume");
-        volume.addControlBinding(DirectControlBinding.createAbsolute(volume, "volume", controller));
+        ControlBinding binding = DirectControlBinding.createAbsolute(volume, "volume", controller);
+
+        assertEquals(2.5,
+                (Double) binding.getValue(ClockTime.fromMillis(500)),
+                0.01);
+
+        Object[] values = new Object[3];
+        binding.getValueArray(0, ClockTime.fromMillis(500), values);
+        assertEquals(0, (Double) values[0], 0.01);
+        assertEquals(2.5, (Double) values[1], 0.01);
+        assertEquals(5, (Double) values[2], 0.01);
+
+        volume.addControlBinding(binding);
         volume.syncValues(0);
-        assertEquals(0, ((Double) volume.get("volume")).doubleValue(), 0.001);
+        assertEquals(0, ((Double) volume.get("volume")), 0.001);
         volume.syncValues(ClockTime.fromMillis(500));
-        assertEquals(2.5, ((Double) volume.get("volume")).doubleValue(), 0.001);
+        assertEquals(2.5, ((Double) volume.get("volume")), 0.001);
         volume.syncValues(ClockTime.fromSeconds(1));
-        assertEquals(5, ((Double) volume.get("volume")).doubleValue(), 0.001);
+        assertEquals(5, ((Double) volume.get("volume")), 0.001);
 
     }