[gstreamer-bugs] [Bug 344013] [oggdemux] use parsers to suck less

[GStreamer (bugzilla.gnome.org)]

http://bugzilla.gnome.org/show_bug.cgi?id=344013


David Schleef <ds> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #123381|0                           |1
        is obsolete|                            |
             Status|NEEDINFO                    |ASSIGNED
         AssignedTo|gstreamer-bugs at lists.source |ds at schleef.org
                   |forge.net                   |
     Ever Confirmed|0                           |1


--- Comment #8 from David Schleef <ds at schleef.org> 2009-08-30 02:34:28 UTC ---
Created an attachment (id=142010)
updated patch

Merged patch with some of my current work and updated for current git.

Patch changes:

 - eliminates the internal decoder inside oggdemux
 - adds code to convert granulepos to timestamps
 - adds code to calculate granules for ogg packets that don't have valid
granulepos values
 - adds code to calculate durations of ogg packets
 - implements the above for theora, vorbis, dirac, speex, celt, flac, ogm

With this patch, oggdemux can independently (without queries or internal
elements) calculate chain duration exactly (it has never been able to do this
correctly before) and fixes #568014.

With minor additional work, oggdemux could add timestamps and durations to
outgoing buffers, estimate the number of packets to seek backward to get to a
key frame, 

Known regressions:

 - calculating vorbis durations is only a good estimate
 - I may have broken skeleton support

-- 
Configure bugmail: http://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.