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

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Tue Apr 29 06:10:26 PDT 2014


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

--- Comment #53 from Gwenole Beauchesne <gb.devel at gmail.com> 2014-04-29 13:10:21 UTC ---
Review of attachment 272786:
 --> (https://bugzilla.gnome.org/review?bug=685215&attachment=272786)

::: gst-libs/gst/codecparsers/gsth264parser.c
@@ +242,3 @@
+      break;
+    default:
+      nalu->extension_type = GST_H264_NAL_EXTENSION_NONE;

Before I forget, default initialization to GST_H264_NAL_EXTENSION_NONE needs to
be moved up before the switch() so that we can have some sensible value for SVC
streams, even if we don't support them. Or, we should correctly initialize that
for SVC NAL units too, and reject them subsequently in parser functions.

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