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 549208 - [mpeg2dec] Should put timestamps on all frames
[mpeg2dec] Should put timestamps on all frames
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-ugly
git master
Other Linux
: Normal normal
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-08-24 16:56 UTC by Sebastian Dröge (slomo)
Modified: 2012-11-25 16:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Sebastian Dröge (slomo) 2008-08-24 16:56:02 UTC 
Hi,
mpeg2dec currently doesn't put timestamps on all frames in some situations. For example mpegdemux/dvddemux don't put timestamps on every packet they output and if they don't put a timestamp on the first packet mpeg2dec will decode it but only start putting valid timestamps on the output after the first timestamp was sent by dvddemux/mpegdemux.

This OTOH confuses encoders like theoraenc and makes it impossible to mux the output into Ogg as there won't be valid GP times.
Comment 1 Edward Hervey 2009-03-06 08:57:28 UTC 
What shall we do then ? Keep decoded buffers (up to a wise limit) until we get an incoming buffer with timestamp and then output properly timestampped buffers ?
Comment 2 Sebastian Dröge (slomo) 2009-03-26 09:28:02 UTC 
Yes, at least that's what thaytan and ds said on IRC
Comment 3 Tim-Philipp Müller 2012-09-26 00:00:32 UTC 
Does the video decoder base class handle this now? If not, should it be moved there?
Comment 4 Tim-Philipp Müller 2012-11-25 16:50:39 UTC 
From the looks of it the video decoder base class tries reasonably hard to ensure this, so closing. Please re-open if this is still an issue.