From 7e2138794f6c41def1d46a198871e30e4ef4c8fe Mon Sep 17 00:00:00 2001 From: George Kiagiadakis Date: Thu, 26 Dec 2013 17:30:42 +0200 Subject: [PATCH] rtpsession: remove unused if branch 1) sources that have sent BYE in the past cannot be senders, since they would have timed out to being receivers in the meantime... 2) sources that have sent BYE are now being removed earlier inside this function --- gst/rtpmanager/rtpsession.c | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/gst/rtpmanager/rtpsession.c b/gst/rtpmanager/rtpsession.c index 1e37406d20..2d1d3078ee 100644 --- a/gst/rtpmanager/rtpsession.c +++ b/gst/rtpmanager/rtpsession.c @@ -3295,16 +3295,9 @@ session_cleanup (const gchar * key, RTPSource * source, ReportData * data) if (data->current_time > btime) { interval = MAX (binterval * 2, 5 * GST_SECOND); if (data->current_time - btime > interval) { - if (source->internal && source->sent_bye) { - /* an internal source is BYE and stopped sending RTP, remove */ - GST_DEBUG ("internal BYE source %08x timed out, last %" - GST_TIME_FORMAT, source->ssrc, GST_TIME_ARGS (btime)); - remove = TRUE; - } else { - GST_DEBUG ("sender source %08x timed out and became receiver, last %" - GST_TIME_FORMAT, source->ssrc, GST_TIME_ARGS (btime)); - sendertimeout = TRUE; - } + GST_DEBUG ("sender source %08x timed out and became receiver, last %" + GST_TIME_FORMAT, source->ssrc, GST_TIME_ARGS (btime)); + sendertimeout = TRUE; } } }