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

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Mon Mar 24 15:27:48 PDT 2014


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

--- Comment #41 from Gwenole Beauchesne <gb.devel at gmail.com> 2014-03-24 22:53:11 UTC ---
Review of attachment 272790:
 --> (https://bugzilla.gnome.org/review?bug=685215&attachment=272790)

::: gst-libs/gst/codecparsers/gsth264parser.c
@@ +950,3 @@
+  if (!frame_packing_extension_flag)
+    goto error;
+  nal_reader_skip (nr, payload_size - (nal_reader_get_pos (nr) - start_pos));

I will introduce a nal_reader_skip_long() function instead. It looks like
nal_reader_skip() is more tailored to nbits <= sizeof(cache) * 8 [64 bits].
Thus, also simplifying the default case in the SEI payloader parser.

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