Changeset 627 for pjproject/trunk

Timestamp:

Jul 26, 2006 11:23:07 AM (18 years ago)

Author:

bennylp

Message:

Fixed rather loud clicking/tick noise when call is established. This was caused by the PLC not properly cleared before it is reused for the next call

Location:

pjproject/trunk/pjmedia/src

Files:

• pjmedia-codec/gsm.c (modified) (2 diffs)
• pjmedia/g711.c (modified) (2 diffs)

pjproject/trunk/pjmedia/src/pjmedia-codec/gsm.c

@@ -344 +344 @@
 {
     struct gsm_data *gsm_data;
+    pj_int16_t frame[160];
+    int i;
 
     PJ_ASSERT_RETURN(factory && codec, PJ_EINVAL);
@@ -352 +354 @@
     /* Close codec, if it's not closed. */
     gsm_codec_close(codec);
+
+    /* Clear left samples in the PLC, since codec+plc will be reused
+     * next time.
+     */
+    for (i=0; i<2; ++i) {
+        pjmedia_zero_samples(frame, PJ_ARRAY_SIZE(frame));
+        pjmedia_plc_save(gsm_data->plc, frame);
+    }
 
     /* Put in the free list. */

pjproject/trunk/pjmedia/src/pjmedia/g711.c

@@ -358 +358 @@
                                       pjmedia_codec *codec )
 {
-    
+    struct g711_private *priv = codec->codec_data;
+    pj_int16_t frame[SAMPLES_PER_FRAME];
+    int i;
+
     PJ_ASSERT_RETURN(factory==&g711_factory.base, PJ_EINVAL);
 
@@ -365 +368 @@
     if (codec->next!=NULL || codec->prev!=NULL) {
         return PJ_EINVALIDOP;
+    }
+
+    /* Clear left samples in the PLC, since codec+plc will be reused
+     * next time.
+     */
+    for (i=0; i<2; ++i) {
+        pjmedia_zero_samples(frame, PJ_ARRAY_SIZE(frame));
+        pjmedia_plc_save(priv->plc, frame);
     }