From e5db2673bdd91a0ef9862ba395c0a9fd88d69bd4 Mon Sep 17 00:00:00 2001 From: Jan Schmidt Date: Mon, 15 Jun 2015 16:12:10 +1000 Subject: [PATCH] matroska-demux: Use gst_video_multiview_guess_half_aspect() Use the gst_video_multiview_guess_half_aspect() utility function to set the half-aspect flag (or not) on stereoscopic frame-packed videos. --- gst/matroska/matroska-demux.c | 22 ++++++++++++++-------- 1 file changed, 14 insertions(+), 8 deletions(-) diff --git a/gst/matroska/matroska-demux.c b/gst/matroska/matroska-demux.c index ad0670fa33..76fd8b830e 100644 --- a/gst/matroska/matroska-demux.c +++ b/gst/matroska/matroska-demux.c @@ -5202,14 +5202,6 @@ gst_matroska_demux_video_caps (GstMatroskaTrackVideoContext * videocontext->display_width * videocontext->pixel_height, videocontext->display_height * videocontext->pixel_width, NULL); } - if (videocontext->multiview_mode != GST_VIDEO_MULTIVIEW_MODE_NONE) { - gst_caps_set_simple (caps, - "multiview-mode", G_TYPE_STRING, - gst_video_multiview_mode_to_caps_string - (videocontext->multiview_mode), "multiview-flags", - GST_TYPE_VIDEO_MULTIVIEW_FLAGSET, videocontext->multiview_flags, - GST_FLAG_SET_MASK_EXACT, NULL); - } if (videocontext->default_fps > 0.0) { gint fps_n, fps_d; @@ -5239,6 +5231,20 @@ gst_matroska_demux_video_caps (GstMatroskaTrackVideoContext * gst_structure_set (structure, "interlace-mode", G_TYPE_STRING, "mixed", NULL); } + if (videocontext->multiview_mode != GST_VIDEO_MULTIVIEW_MODE_NONE) { + if (gst_video_multiview_guess_half_aspect (videocontext->multiview_mode, + videocontext->pixel_width, videocontext->pixel_height, + videocontext->display_width * videocontext->pixel_height, + videocontext->display_height * videocontext->pixel_width)) { + videocontext->multiview_flags |= GST_VIDEO_MULTIVIEW_FLAGS_HALF_ASPECT; + } + gst_caps_set_simple (caps, + "multiview-mode", G_TYPE_STRING, + gst_video_multiview_mode_to_caps_string + (videocontext->multiview_mode), "multiview-flags", + GST_TYPE_VIDEO_MULTIVIEW_FLAGSET, videocontext->multiview_flags, + GST_FLAG_SET_MASK_EXACT, NULL); + } caps = gst_caps_simplify (caps); }