mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2025-04-26 05:16:13 +00:00
Code Issues 1 Projects Releases Wiki Activity

rtspsrc: fix some comments, remove property check

Browse source 
Fix some comments, clarify some FIXMEs
Remove the on-ntp-stop signal check now that the jitterbuffer is in
-good and we know that it supports this signal.
This commit is contained in:
Wim Taymans 2009-12-24 22:23:01 +01:00 committed by Wim Taymans
parent 3c0f18d765
commit a65240d1c1
1 changed files with 12 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
gst/rtsp/gstrtspsrc.c
View file

@ -814,7 +814,11 @@ gst_rtspsrc_create_stream (GstRTSPSrc * src, GstSDPMessage * sdp, gint idx)
gst_rtspsrc_collect_bandwidth (src, sdp, media, stream); gst_rtspsrc_collect_bandwidth (src, sdp, media, stream);
/* we must have a payload. No payload means we cannot create caps */ /* we must have a payload. No payload means we cannot create caps */
/* FIXME, handle multiple formats. */ /* FIXME, handle multiple formats. The problem here is that we just want to
* take the first available format that we can handle but in order to do that
* we need to scan for depayloader plugins. Scanning for payloader plugins is
* also suboptimal because the user maybe just wants to save the raw stream
* and then we don't care. */
if ((payload = gst_sdp_media_get_format (media, 0))) { if ((payload = gst_sdp_media_get_format (media, 0))) {
stream->pt = atoi (payload); stream->pt = atoi (payload);
/* convert caps */ /* convert caps */
@ -1203,7 +1207,11 @@ gst_rtspsrc_media_to_caps (gint pt, const GstSDPMedia * media)
if (valpos) { if (valpos) {
/* we have a '=' and thus a value, remove the '=' with \0 */ /* we have a '=' and thus a value, remove the '=' with \0 */
*valpos = '\0'; *valpos = '\0';
/* value is everything between '=' and ';'. FIXME, strip? */ /* value is everything between '=' and ';'. We split the pairs at ;
* boundaries so we can take the remainder of the value. Some servers
* put spaces around the value which we strip off here. Alternatively
* we could strip those spaces in the depayloaders should these spaces
* actually carry any meaning in the future. */
val = g_strstrip (valpos + 1); val = g_strstrip (valpos + 1);
} else { } else {
/* simple <param>;.. is translated into <param>=1;... */ /* simple <param>;.. is translated into <param>=1;... */
@ -2091,13 +2099,8 @@ gst_rtspsrc_stream_configure_manager (GstRTSPSrc * src, GstRTSPStream * stream,
src); src);
g_signal_connect (src->session, "on-timeout", (GCallback) on_timeout, g_signal_connect (src->session, "on-timeout", (GCallback) on_timeout,
src); src);
/* FIXME: remove this once the rtpjitterbuffer is in -good */ g_signal_connect (src->session, "on-npt-stop", (GCallback) on_npt_stop,
if (g_signal_lookup ("on-npt-stop", G_OBJECT_TYPE (src->session)) != 0) { src);
g_signal_connect (src->session, "on-npt-stop", (GCallback) on_npt_stop,
src);
} else {
GST_INFO_OBJECT (src, "skipping on-npt-stop handling, not implemented");
}
} }
/* we stream directly to the manager, get some pads. Each RTSP stream goes /* we stream directly to the manager, get some pads. Each RTSP stream goes