[Bug 722760] Add VP8 parser in codecparsers submodule

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Fri Jan 24 01:35:51 PST 2014


https://bugzilla.gnome.org/show_bug.cgi?id=722760
  GStreamer | gst-plugins-bad | 1.x

--- Comment #15 from Zhao, Halley <halley.zhao at intel.com> 2014-01-24 09:35:48 UTC ---
thanks slomo; patches have been update.

1. "init the frame_hdr with 0", 
there are two parts of parameter in frame_hdr, one is parsed from current
frame; another is inherited from previous frame(multi_frame_data) which current
frame may update them or may not. 
so, it's better: decoder init the frame_hdr to 0, then init multi_frame_data.

2. make range decoder to be stack allocatable.
now, I follow the GstByteReader implementation. 
vp8 parser needn't know the data structure of GstRangeDecoder.

3. what's the value to have a vp8 parser element?
vp8 frame size is always decided by its container (webm or ivf), there is no
sync code in vp8 stream. when will a vp8 parser element is needed?

i don't have schedule for vp9 parser yet.

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