[Bug 740899] video-info: Add plane size information

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=740899
  GStreamer | gst-plugins-base | git

--- Comment #3 from Nicolas Dufresne (stormer) <nicolas.dufresne at collabora.co.uk> 2014-12-01 15:04:22 UTC ---
(In reply to comment #2)
> (From update of attachment 291801 [details])
> Looks good to me but I don't like that we get rid of all the padding ;)

Do you have a way around that ?

> 
> Isn't the same also needed for GstVideoMeta?

Size and plane_size[] are only useful for allocation. When VideoMeta
(default_map) maps a memory, it currently uses find_memory with the offset and
a size of 1. Passing plane_size for validating that plane is large enough would
not work, as there is no indication how much data has been added for padding
before the offset. Using GStreamer default plane sizes for this width and
height may not work either as our default includes some padding at the end of
planes to enhance alignement for system memory operations when allocating
multiple planes in a single buffer.

Though, adding plane_size[] (along with missing size) to VideoMeta could be
nice. As right now, the updated VideoInfo endup with invalid size, hence
plane_size[] values. But this is a different issue that also require a slight
API juggling.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.