Re: [0.11] gstreamer: controller: remove GstValueArray

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Re: [0.11] gstreamer: controller: remove GstValueArray

Sebastian Dröge
On Mo, 2011-12-05 at 23:38 -0800, Stefan Kost wrote:

> Module: gstreamer
> Branch: 0.11
> Commit: 4b29d4f29ac856f1903fcaebda136164ca8323d9
> URL:    http://cgit.freedesktop.org/gstreamer/gstreamer/commit/?id=4b29d4f29ac856f1903fcaebda136164ca8323d9
>
> Author: Stefan Sauer <[hidden email]>
> Date:   Tue Dec  6 08:35:57 2011 +0100
>
> controller: remove GstValueArray
>
> Instead pass the values as arguments. This simplifies that code and helps
> bindings.
Not much unfortunately. The "gpointer *" argument should probably be a
"gpointer", right? But this doesn't help bindings much because they
still can't know the type behind this pointer. The only thing that helps
bindings here would either be _get_int_value_array(),
_get_float_value_array(), etc. functions or using GValueArray.

_______________________________________________
gstreamer-devel mailing list
[hidden email]
http://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

signature.asc (205 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [0.11] gstreamer: controller: remove GstValueArray

Stefan Sauer
On 12/06/2011 09:11 AM, Sebastian Dröge wrote:

> On Mo, 2011-12-05 at 23:38 -0800, Stefan Kost wrote:
>> Module: gstreamer
>> Branch: 0.11
>> Commit: 4b29d4f29ac856f1903fcaebda136164ca8323d9
>> URL:    http://cgit.freedesktop.org/gstreamer/gstreamer/commit/?id=4b29d4f29ac856f1903fcaebda136164ca8323d9
>>
>> Author: Stefan Sauer <[hidden email]>
>> Date:   Tue Dec  6 08:35:57 2011 +0100
>>
>> controller: remove GstValueArray
>>
>> Instead pass the values as arguments. This simplifies that code and helps
>> bindings.
> Not much unfortunately. The "gpointer *" argument should probably be a
> "gpointer", right?
Fixed.

Stefan
_______________________________________________
gstreamer-devel mailing list
[hidden email]
http://lists.freedesktop.org/mailman/listinfo/gstreamer-devel