[Bug 722760] Add VP8 parser in codecparsers submodule

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Thu Apr 3 04:53:40 PDT 2014


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

Gwenole Beauchesne <gb.devel> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #273428|0                           |1
        is obsolete|                            |

--- Comment #35 from Gwenole Beauchesne <gb.devel at gmail.com> 2014-04-03 11:53:33 UTC ---
Created an attachment (id=273522)
 View: https://bugzilla.gnome.org/attachment.cgi?id=273522
 Review: https://bugzilla.gnome.org/review?bug=722760&attachment=273522

codecparsers: add VP8 bitstream parser

Changes:
- More documentation
- Renamed a few things to better match the reality
- Fixed a couple of bugs, in particular when default values are to be used
- Calculated all partition sizes, including the last one
- Removed frame_data from frame header struct, this is an invitation to
dangling pointers. Rather, pass down the persistent state as an additional
argument to gst_vp8_parse_frame_hdr().

I am not fully satisfied yet of that version:
- Some minor improvements are needed for the range decoder state
- I would like to add a header_size in bits at the end of the frame_hdr too

There is also another patch to calculate PAR.

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