Revert "rtpjitterbuffer: If we have an immediate timeout, don't try to find an earlie...

This reverts commit 0c21cd7177ea883c710999147ddcedb19004d182.

If we have multiple immediate timers, we want to first handle the one with the
lowest sequence number... which would be broken now.

Instead of this we should just use a GSequence for the timers, and have them
sorted first by timestamp, and for equal timestamps by sequence number. Then
we would always only have to take the very first timer from the list and never
have to look at any others.

diff --git a/gst/rtpmanager/gstrtpjitterbuffer.c b/gst/rtpmanager/gstrtpjitterbuffer.c
index 306f04a..8f9c51f 100644 (file)
--- a/gst/rtpmanager/gstrtpjitterbuffer.c
+++ b/gst/rtpmanager/gstrtpjitterbuffer.c
@@ -3315,10 +3315,6 @@ wait_next_timeout (GstRtpJitterBuffer * jitterbuffer)
         GST_DEBUG_OBJECT (jitterbuffer, "new best %d", i);
         timer = test;
         timer_timeout = test_timeout;
-
-        /* Immediate timeout */
-        if (test_timeout == -1)
-          break;
       }
     }
     if (timer && !priv->blocked) {