[Bug 741030] theoradec: Sets video-meta width/height from padded values

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Wed Dec 17 07:38:44 PST 2014


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

--- Comment #2 from Nicolas Dufresne (stormer) <nicolas.dufresne at collabora.co.uk> 2014-12-17 15:38:42 UTC ---
No, it's being discussed. My own reflection was that having padded width/height
around would allow receiver to validate the crop area instead of going out of
bound if the crop is wrong.

At the same time, theoradec uses 16x16 alignment and avoid chroma shifting
issues where needed. Base on these documented constraints we could replace this
code with VideoAlignment as being done in gst-libav. The benefit is that
downstream don't need to know there is padding, all they need to handle is the
stride.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- 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