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 750111 - rtsp-media: emit signal for preparing dynamic payloads
rtsp-media: emit signal for preparing dynamic payloads
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-rtsp-server
git master
Other Linux
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks: 767021
 
 
Reported: 2015-05-29 15:22 UTC by George Kiagiadakis
Modified: 2018-11-03 15:38 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (2.17 KB, patch)
2015-05-29 15:22 UTC, George Kiagiadakis
none Details | Review
rtsp-media: emit signal for preparing dynamic payloads (2.81 KB, patch)
2017-02-23 11:11 UTC, George Kiagiadakis
none Details | Review

Description George Kiagiadakis 2015-05-29 15:22:43 UTC
Created attachment 304268 [details] [review]
patch

When using dynamic payloads, it is useful to know when rtsp-media has connected to the pad-added signal, so that the application code can continue to add those pads. If the dynpay element is a decodebin, the pads are always added after the element changes state to PAUSED, which is set after connecting the pad-added signal internally and therefore it works fine. However, there can be other cases where the application needs to add the pads externally and therefore it needs to know when is the right moment to do so.

This patch adds a "preparing" signal that is emitted at the end of start_prepare, and compliments the "prepared" and "unprepared" signals.
Comment 1 George Kiagiadakis 2015-11-09 17:00:30 UTC
Any thoughts on this?
Comment 2 Sebastian Dröge (slomo) 2016-10-20 10:40:14 UTC
Review of attachment 304268 [details] [review]:

::: gst/rtsp-server/rtsp-media.h
@@ +158,3 @@
   void            (*removed_stream)  (GstRTSPMedia *media, GstRTSPStream * stream);
 
+  void            (*preparing)       (GstRTSPMedia *media);

This breaks ABI, has to go at the very end and remove some padding
Comment 3 George Kiagiadakis 2017-02-23 11:11:13 UTC
Created attachment 346559 [details] [review]
rtsp-media: emit signal for preparing dynamic payloads

Here is a fixed (wrt ABI issues) version of the patch, which applies on top of latest master.
Comment 4 GStreamer system administrator 2018-11-03 15:38:42 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-rtsp-server/issues/11.