[Bug 760270] videoparse: add support of padded video

[GStreamer (GNOME Bugzilla)]

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

--- Comment #36 from Aurélien Zanelli <aurelien.zanelli at parrot.com> ---
(In reply to Nicolas Dufresne (stormer) from comment #33)
> Created attachment 319957 [details] [review]
> videoparse: Fix framesize calculation
> 
> When the framesize is not specified, we try and calculate a size from
> the strides and offset information. This was done with the sum of
> offsets + the size of the last frame. That is just wrong method. We also
> need to account for video meta that may be flipping two planes. An
> example is if you convert I420 to YV12 by flipping the two last offsets.

I don't set the patch as accepted as I don't know if I'm allowed to do this but
patch looks good. 
There is just 'planesize' which may be a confusion name though.

I tested it with I420 and Y42B with default and custom stride and it's ok.
However flipping offsets doesn't work at all since user provided offsets are
checked against provided format defaults offsets which are sorted from small to
large. Hence offsets will likely be ignored in this case.

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