Changeset 2672 for pjproject/trunk/pjmedia/src/pjmedia/jbuf.c

Timestamp:

May 1, 2009 3:13:01 PM (14 years ago)

Author:

nanang

Message:

Ticket #762: Fixed jitter buffer in handling DTX with relative short period of 'keep alive', e.g: Speex DTX seems to periodically send a keep alive frame every 20 silence frames.

File:

• pjproject/trunk/pjmedia/src/pjmedia/jbuf.c (modified) (3 diffs)

pjproject/trunk/pjmedia/src/pjmedia/jbuf.c

@@ -316 +316 @@
     assert(frame_size <= framelist->frame_size);
 
-    /* the first ever frame since inited/resetted. */
-    if (framelist->origin == INVALID_OFFSET)
-        framelist->origin = index;
-
     /* too late or duplicated or sequence restart */
     if (index < framelist->origin) {
@@ -331 +327 @@
     }
 
+    /* if jbuf is empty, just reset the origin */
+    if (framelist->size == 0) {
+        framelist->origin = index;
+    }
+
     /* get distance of this frame to the first frame in the buffer */
     distance = index - framelist->origin;
@@ -342 +343 @@
             distance = 0;
         } else {
-            if (framelist->size == 0) {
-                /* if jbuf is empty, process the frame */
-                framelist->origin = index;
-                distance = 0;
-            } else {
-                /* otherwise, reject the frame */
-                return PJ_ETOOMANY;
-            }
+            /* otherwise, reject the frame */
+            return PJ_ETOOMANY;
         }
     }