[Bug 794822] rtsp-server: add API to enable retransmission requests

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Fri Mar 30 12:39:06 UTC 2018


https://bugzilla.gnome.org/show_bug.cgi?id=794822

--- Comment #5 from Mathieu Duponchelle <mduponchelle1 at gmail.com> ---
(In reply to Sebastian Dröge (slomo) from comment #3)
> Review of attachment 370315 [details] [review]:
> 
> Looks mostly good to me
> 
> ::: gst/rtsp-server/rtsp-media.c
> @@ +163,3 @@
>  #define DEFAULT_STOP_ON_DISCONNECT TRUE
>  
> +#define DEFAULT_DO_RETRANSMISSION FALSE
> 
> Also needs a GObject property for consistency I guess

rtx-time doesn't expose a property, I can add one but then one will also be
needed in media-factory

> 
> ::: gst/rtsp-server/rtsp-stream.c
> @@ +2369,3 @@
> +  stream->priv->rtxreceive = gst_element_factory_make ("rtprtxreceive",
> NULL);
> +  pt_map = gst_structure_new_empty ("application/x-rtp-pt-map");
> +  g_hash_table_foreach (stream->priv->ptmap, (GHFunc) add_rtx_pt, pt_map);
> 
> This might have to be updated whenever the ptmap changes, or do we just not
> support that?

I'm not sure whether a new ANNOUNCE in the RECORD case is supported by
rtsp-server, but I'll update stream_set_pt_map anyway :)

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list