[Bug 722760] Add VP8 parser in codecparsers submodule

[GStreamer (bugzilla.gnome.org)]

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

Sebastian Dröge (slomo) <slomo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |slomo at coaxion.net

--- Comment #16 from Sebastian Dröge (slomo) <slomo at coaxion.net> 2014-01-24 12:26:37 UTC ---
See comment 12 and comment 13, otherwise I think this is good to go.

(In reply to comment #15)
> 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.

Should be documented but otherwise ok :)

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

Not sure when it would be needed, probably never... or for broken containers
that don't mark keyframes properly, or for extracting further information from
the VP8 bitstream (droppable frames, no-display frames, ...).

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