rtpsource: add comment to explain why probation queue is not always cleared
authorAntonio Ospite <antonio.ospite@collabora.com>
Wed, 6 Mar 2019 08:52:45 +0000 (09:52 +0100)
committerAntonio Ospite <antonio.ospite@collabora.com>
Tue, 2 Apr 2019 11:03:56 +0000 (13:03 +0200)
gst/rtpmanager/rtpsource.c

index d712fc6..c381b4f 100644 (file)
@@ -1133,7 +1133,14 @@ update_receiver_stats (RTPSource * src, RTPPacketInfo * pinfo,
           goto done;
         }
       } else {
-        /* unexpected seqnum in probation */
+        /* unexpected seqnum in probation
+         *
+         * There is no need to clean the queue at this point because the
+         * invalid packets in the queue are not going to be pushed as we are
+         * still in probation, and some cleanup will be performed at future
+         * probation attempts anyway if there are too many old packets in the
+         * queue.
+         */
         goto probation_seqnum;
       }
     } else if (delta >= 0 && delta < max_dropout) {