Changeset 2436 for pjproject/branches/projects/aps-direct/pjmedia/include/pjmedia/port.h

Timestamp:

Jan 29, 2009 8:06:28 PM (15 years ago)

Author:

nanang

Message:

• Added pjmedia passthrough codec.
• Added field format in codec param info.
• Updated pjmedia_frame_ext_get_subframe to have index range checking.
• Added API docs on pjmedia_frame_ext routines.

File:

• pjproject/branches/projects/aps-direct/pjmedia/include/pjmedia/port.h (modified) (2 diffs)

pjproject/branches/projects/aps-direct/pjmedia/include/pjmedia/port.h

@@ -285 +285 @@
 
 
-/* Append one subframe to the frame_ext */
+/**
+ * Append one subframe to #pjmedia_frame_ext.
+ *
+ * @param frm               The #pjmedia_frame_ext.
+ * @param src               Subframe data.
+ * @param bitlen            Lenght of subframe, in bits.
+ * @param samples_cnt       Number of audio samples in subframe.
+ */
 PJ_INLINE(void) pjmedia_frame_ext_append_subframe(pjmedia_frame_ext *frm,
                                                   const void *src,
@@ -311 +318 @@
 }
 
-/* Get the pointer and length of the n-th subframe */
+/**
+ * Get a subframe from #pjmedia_frame_ext.
+ *
+ * @param frm               The #pjmedia_frame_ext.
+ * @param n                 Subframe index, zero based.
+ *
+ * @return                  The n-th subframe, or NULL if n is out-of-range.
+ */
 PJ_INLINE(pjmedia_frame_ext_subframe*) 
           pjmedia_frame_ext_get_subframe(const pjmedia_frame_ext *frm,
                                          unsigned n)
 {
-    pj_uint8_t *p;
-    unsigned i;
-    pjmedia_frame_ext_subframe *tmp;
-
-    pj_assert(n < frm->subframe_cnt);
-
-    p = (pj_uint8_t*)frm + sizeof(pjmedia_frame_ext);
-    for (i = 0; i < n; ++i) {   
-        tmp = (pjmedia_frame_ext_subframe*) p;
-        p += tmp->bitlen / 8;
-        if (tmp->bitlen % 8)
-            ++p;
+    pjmedia_frame_ext_subframe *sf = NULL;
+
+    if (n < frm->subframe_cnt) {
+        pj_uint8_t *p;
+        unsigned i;
+
+        p = (pj_uint8_t*)frm + sizeof(pjmedia_frame_ext);
+        for (i = 0; i < n; ++i) {       
+            sf = (pjmedia_frame_ext_subframe*) p;
+            p += sf->bitlen / 8;
+            if (sf->bitlen % 8)
+                ++p;
+        }
+        
+        sf = (pjmedia_frame_ext_subframe*) p;
     }
-    
-    tmp = (pjmedia_frame_ext_subframe*) p;
-    return tmp;
+
+    return sf;
 }