After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 308827 - ogg muxer in 0.8 muxes wrong
ogg muxer in 0.8 muxes wrong
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins
0.8.9
Other Linux
: Normal normal
: 0.8.10
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-06-23 19:44 UTC by Thomas Vander Stichele
Modified: 2005-06-24 17:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
push out pages sorted by timestamp matching page granulepos (25.93 KB, patch)
2005-06-23 19:46 UTC, Thomas Vander Stichele
none Details | Review
updated patch (28.54 KB, patch)
2005-06-23 23:49 UTC, Thomas Vander Stichele
none Details | Review

Description Thomas Vander Stichele 2005-06-23 19:44:57 UTC
it muxes based on the timestamp matching the granulepos of the *first packet*
going on a page.  it should mux based on timestamp matching the granulepos set
on a completed page.
Comment 1 Thomas Vander Stichele 2005-06-23 19:46:57 UTC
Created attachment 48228 [details] [review]
push out pages sorted by timestamp matching page granulepos

when pages get output, put them on a per-pad queue.  dequeue regularly, taking
the page with the lowest timestamp, but only if all per-pad queues have pages. 
push that page.
Comment 2 Thomas Vander Stichele 2005-06-23 23:49:33 UTC
Created attachment 48255 [details] [review]
updated patch

updated patch; correctly handles -1 granulepos, and fixes wrong timestamping
when packets span pages
Comment 3 Thomas Vander Stichele 2005-06-24 17:37:03 UTC
cleaned up and commited to cvs, fixed.