[Bug 780053] rawvideoparse: Stride/Offset no longer usable from gst-launch

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=780053

--- Comment #9 from Nicolas Dufresne (stormer) <nicolas at ndufresne.ca> ---
(In reply to Sebastian Dröge (slomo) from comment #8)
> Review of attachment 348037 [details] [review]:
> 
> Is this fixing anything or just changing the API?

It makes it usable from gst-launch, which really matter for a development tool
like the raw parser.

> 
> I don't think having only a GstValueArray property is a good idea. This
> makes it nearly impossible to use from bindings. See all the discussions we
> had about audiomixmatrix, where we now ended up with two properties: one
> GValueArray for bindings, one GstValueArray for gst-launch. It all sucks
> really, but this seems to be the least bad approach to make usage easy.

Or could I just go back to using a string, like I did when I added strides and
offset in the origin videoparse element ?

(p.s. the feature is still broken in some cases, it will set the stride, and
then update it wrongly with a larger value, additionally, it fails to set the
GstVideoMeta from time to time, it does not detect if downstream supports
videometa. There a lot of work to fix this element, the unit test is hardcoded
to a fixed resolution and Y444)

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.