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 322347 - GstBaseRtpDepayload timestamps are wring
GstBaseRtpDepayload timestamps are wring
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: 0.10.1
Assigned To: Philippe Khalaf
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-11-24 17:14 UTC by Sebastien Cote
Modified: 2005-12-18 01:59 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed fix (1.08 KB, patch)
2005-11-24 17:16 UTC, Sebastien Cote
none Details | Review

Description Sebastien Cote 2005-11-24 17:14:42 UTC
In GstBaseRtpDepayload, the new segment is sent with the RTP timestamp of the
first buffer. The first buffer's timestamp (and the
following ones) is then updated to add the queue
length.

I think the queue length should also be added on the
new segment's timestamp. Then, the new segment and the
first buffer will have the same timestamp.

Or, as someone mentioned on the mailing-list, we could not add the queue delay
to the buffer's timestamp.
Comment 1 Sebastien Cote 2005-11-24 17:16:07 UTC
Created attachment 55194 [details] [review]
proposed fix

Add the queue delay to the new_segment event.