[Bug 735379] mpeg2dec: use stride information from GstVideoFrame

[GStreamer (GNOME Bugzilla)]

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

Nicolas Dufresne (stormer) <nicolas.dufresne at collabora.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nicolas.dufresne at collabora.
                   |                            |co.uk

--- Comment #7 from Nicolas Dufresne (stormer) <nicolas.dufresne at collabora.co.uk> ---
I think all the approaches so far are failing because they don't address the
problem too well. From more reading, mpeg2dec requires allocation to be 16bytes
aligned and this seems to be covered. But then it also require a stride
alignment of 16, and a pixel height alignment of 16. Finally this patch address
the issue that it required uniformed strides.

If we can cover all this, that would be much better. The decide allocation
method is also very off. This decoder is very similar to libav video decoders,
except that it does not offer a method to copy crop. Fortunately, it's always a
top left align crop, which we can handle with video_frame_copy.

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