Changeset 3879 for pjproject

Timestamp:

Oct 31, 2011 3:48:15 PM (13 years ago)

Author:

nanang

Message:

Fix #1410:

• Fixed bug in ipp_codecs.c pack_amr() in setting up AMR-WB frame size, i.e: using AMR-NB frame size table instead of AMR-WB's.
• Updated AMR-WB default setting to use two frames-per-packet.

File:

• pjproject/branches/1.x/pjmedia/src/pjmedia-codec/ipp_codecs.c (modified) (3 diffs)

pjproject/branches/1.x/pjmedia/src/pjmedia-codec/ipp_codecs.c

@@ -248 +248 @@
 #   if PJMEDIA_HAS_INTEL_IPP_CODEC_AMRWB
     {1, "AMR-WB",   PJMEDIA_RTP_PT_AMRWB,     &USC_AMRWB_Fxns,  16000, 1, 320,
-                    15850, 23850, 1, 1, 1, 
+                    15850, 23850, 2, 1, 1, 
                     &predecode_amr, &parse_amr, &pack_amr,
                     {1, {{{"octet-align", 11}, {"1", 1}}} }
@@ -510 +510 @@
     pj_uint8_t SID_FT;
     pjmedia_codec_amr_pack_setting *setting;
+    const pj_uint8_t *framelen_tbl;
 
     setting = &((amr_settings_t*)codec_data->codec_setting)->enc_setting;
+    framelen_tbl = setting->amr_nb? pjmedia_codec_amrnb_framelen:
+                                    pjmedia_codec_amrwb_framelen;
 
     SID_FT = (pj_uint8_t)(setting->amr_nb? 8 : 9);
@@ -536 +539 @@
         frames[nframes].buf = r + 2;
         frames[nframes].size = info->frame_type <= SID_FT ?
-                               pjmedia_codec_amrnb_framelen[info->frame_type] :
-                               0;
+                               framelen_tbl[info->frame_type] : 0;
 
         r += frames[nframes].size + 2;