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 527386 - [PATCH] Adding timestamps to flutsmux
[PATCH] Adding timestamps to flutsmux
Status: RESOLVED NOTGNOME
Product: GStreamer
Classification: Platform
Component: dont know
git master
Other All
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-04-10 17:13 UTC by Gustaf Räntilä
Modified: 2008-04-10 21:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
The patch (1.24 KB, patch)
2008-04-10 17:15 UTC, Gustaf Räntilä
none Details | Review

Description Gustaf Räntilä 2008-04-10 17:13:15 UTC
Currently, the flutsmux element of gstreamer does not add timestamps to its output buffers. This makes it impossible to correctly stream TS muxed data, for instance over udp (with the udpsink element).

The patch attached to this bug solves this problem by storing the last timestamp before writing data to libtsmux (only for the pcr_stream), and then using that timestamp in the function "new_packet_cb" which libtsmux calls when it has a finished TS packet. It is here, that I store the timestamp on the output buffer.

This has been tested and works satisfyingly.
Comment 1 Gustaf Räntilä 2008-04-10 17:15:03 UTC
Created attachment 109013 [details] [review]
The patch
Comment 2 René Stadler 2008-04-10 20:53:37 UTC
The Fluendo elements have their own bug tracker and everything, you need to create a ticket against gst-fluendo-mpegmux over there:

https://core.fluendo.com/gstreamer/trac/wiki

https://core.fluendo.com/gstreamer/trac/newticket
Comment 3 Gustaf Räntilä 2008-04-10 21:09:07 UTC
Ok thanks, got pointed here from #gstreamer.