[Bug 611157] video: API to signal stereoscopic and multiview video

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #69 from sreerenj <bsreerenj at gmail.com> 2013-04-30 07:43:27 UTC ---
(In reply to comment #62)
> Review of attachment 241094 [details]:
> 
> ::: gst-libs/gst/video/gstvideodecoder.h
> @@ +367,3 @@
> 
> +void             gst_video_decoder_set_stereo_info (GstVideoDecoder *dec,
> +                                                    GstStereoVideoInfo
> *sv_info);
> 
> This should probably be part of the GstVideoCodecState

gst_video_stereo_info_init(), gst_video_stereo_info_from_caps and
gst_video_stereo_buffer_add_video_meta() are only for BaseVideoDecoder. No
other element needs to use this and not even for subclass implementation(As far
as i know). 

The only thing which the subclass implementation needs to do is: invoke
gst_video_decoder_set_stereo_info() if it parsed any stereo-video info.

So I thought it is better to keep a separate structure instead of messing up
with the current *Video utility structures existing.

-- 
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.