[Bug 685215] codecparsers: h264: Add initial MVC parser

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #28 from Gwenole Beauchesne <gb.devel at gmail.com> 2014-03-24 13:54:04 UTC ---
Review of attachment 238381:
 --> (https://bugzilla.gnome.org/review?bug=685215&attachment=238381)

Fixed locally. Several issues in there. BTW, it would be interesting to list
any sample that exhibit this SEI message. Thanks. :)

BTW, this patch is non-essential for the whole MVC codecparser series.

::: gst-libs/gst/codecparsers/gsth264parser.c
@@ +1210,3 @@
+  READ_UINT8 (nr, info->next_frame_is_second_view_flag, 1);
+  READ_UINT8 (nr, info->left_view_self_contained_flag, 1);
+  READ_UINT8 (nr, info->right_view_self_contained_flag, 1);

Missed field_views_flag condition for top_field_is_left_view_flag,
current_frame_is_left_view_flag, next_frame_is_second_view_flag.

Besides, using the _unchecked variant is possible and desirable.

::: gst-libs/gst/codecparsers/gsth264parser.h
@@ +875,3 @@
     GstH264BufferingPeriod buffering_period;
     GstH264PicTiming pic_timing;
+    GstH264StereoVideoInfo stereo_info;

stereo_video_info to match H.264 spec.

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