[Bug 760270] videoparse: add support of padded video

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Thu Jan 28 10:19:01 PST 2016


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

--- Comment #37 from Aurélien Zanelli <aurelien.zanelli at parrot.com> ---
(In reply to Nicolas Dufresne (stormer) from comment #34)
> That fixes it for me, need more testing (I'm a bit in a hurry now). Another
> issue is that we have no equivalent for framesize_set, so if we set a stride
> or an offset, we can no longer also add a framesize (it get overwritten).
> This can be solved by overriding the property, and then proxying to the base
> class with the appropriate method.

User provided framesize is set after size update from strides/offset to video
info struct (only if bigger). Then when we set the base class framesize, we
read back the value from info so it takes the user value if one has been set
and is valid.
Confusing thing is that I updated directly vp->info.size instead of info->size
(info point to &vp->info). I could make a patch to replace vp->info by info to
make it clearer.

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


More information about the gstreamer-bugs mailing list