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

Timestamp:

Jan 31, 2008 8:11:30 AM (16 years ago)

Author:

bennylp

Message:

Ticket #464: jitter buffer should return frame length information

File:

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

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

@@ -35 +35 @@
     char        *flist_buffer;
     int         *flist_frame_type;
+    pj_size_t   *flist_content_len;
     unsigned     flist_frame_size;
     unsigned     flist_max_count;
@@ -110 +111 @@
                                 framelist->flist_max_count);
 
+    framelist->flist_content_len = (pj_size_t*)
+        pj_pool_zalloc(pool, sizeof(framelist->flist_content_len[0]) * 
+                                framelist->flist_max_count);
+
     framelist->flist_empty = 1;
 
@@ -135 +140 @@
 
 static pj_bool_t jb_framelist_get(jb_framelist_t *framelist,
-                                  void *frame,
+                                  void *frame, pj_size_t *size,
                                   pjmedia_jb_frame_type *p_type) 
 {
@@ -145 +150 @@
         *p_type = (pjmedia_jb_frame_type) 
                   framelist->flist_frame_type[framelist->flist_head];
+        *size   = framelist->flist_content_len[framelist->flist_head];
 
         pj_bzero(framelist->flist_buffer + 
@@ -151 +157 @@
         framelist->flist_frame_type[framelist->flist_head] = 
             PJMEDIA_JB_MISSING_FRAME;
+        framelist->flist_content_len[framelist->flist_head] = 0;
 
         framelist->flist_origin++;
@@ -195 +202 @@
                   PJMEDIA_JB_MISSING_FRAME,
                   step1*sizeof(framelist->flist_frame_type[0]));
+        pj_bzero(framelist->flist_content_len+framelist->flist_head,
+                  step1*sizeof(framelist->flist_content_len[0]));
 
         if (step2) {
@@ -202 +211 @@
                       PJMEDIA_JB_MISSING_FRAME,
                       step2*sizeof(framelist->flist_frame_type[0]));
+            pj_bzero (framelist->flist_content_len,
+                      step2*sizeof(framelist->flist_content_len[0]));
         }
 
@@ -258 +269 @@
 
     framelist->flist_frame_type[where] = PJMEDIA_JB_NORMAL_FRAME;
+    framelist->flist_content_len[where] = frame_size;
 
     return PJ_TRUE;
@@ -511 +523 @@
                                      char *p_frame_type)
 {
+    pj_size_t size;
+
+    pjmedia_jbuf_get_frame2(jb, frame, &size, p_frame_type);
+}
+
+/*
+ * Get frame from jitter buffer.
+ */
+PJ_DEF(void) pjmedia_jbuf_get_frame2(pjmedia_jbuf *jb, 
+                                     void *frame, 
+                                     pj_size_t *size,
+                                     char *p_frame_type)
+{
     pjmedia_jb_frame_type ftype;
 
@@ -531 +556 @@
             *p_frame_type = PJMEDIA_JB_ZERO_PREFETCH_FRAME;
 
+        *size = 0;
+
         return;
     }
 
     /* Retrieve a frame from frame list */
-    if (jb_framelist_get(&jb->jb_framelist,frame,&ftype) == PJ_FALSE) {
+    if (jb_framelist_get(&jb->jb_framelist,frame,size,&ftype) == PJ_FALSE) {
         /* Can't return frame because jitter buffer is empty! */
         pj_bzero(frame, jb->jb_frame_size);
         *p_frame_type = PJMEDIA_JB_ZERO_EMPTY_FRAME;
+        *size = 0;
 
         return;