shithub: opus

Download patch

ref: 11f01729e636b0f8a8c6258cfbaa0ddf102da64a
parent: 06ee7f9083335128a0ad72c8dcbdad834e823e7b
author: Jean-Marc Valin <[email protected]>
date: Sat Dec 8 20:19:36 EST 2007 

Some cleaning up, a few less warnings and the decoder no longer does an
infinitete loop in Laplace decoding when the data is corrupted.


--- a/configure.ac
+++ b/configure.ac
@@ -6,7 +6,7 @@
 
 CELT_MAJOR_VERSION=0
 CELT_MINOR_VERSION=0
-CELT_MICRO_VERSION=0
+CELT_MICRO_VERSION=1
 CELT_EXTRA_VERSION=
 CELT_VERSION=$CELT_MAJOR_VERSION.$CELT_MINOR_VERSION.$CELT_MICRO_VERSION$CELT_EXTRA_VERSION
 
--- a/libcelt/bands.c
+++ b/libcelt/bands.c
@@ -155,7 +155,7 @@
    
    for (i=0;i<m->nbEBands;i++)
    {
-      int q, id;
+      int q;
       q = m->nbPulses[i];
       if (q>0) {
          float n = sqrt(B*(eBands[i+1]-eBands[i]));
@@ -185,7 +185,7 @@
    
    for (i=0;i<m->nbEBands;i++)
    {
-      int q, id;
+      int q;
       q = m->nbPulses[i];
       if (q>0) {
          float n = sqrt(B*(eBands[i+1]-eBands[i]));
--- a/libcelt/celt.c
+++ b/libcelt/celt.c
@@ -390,7 +390,7 @@
    celt_free(st);
 }
 
-int celt_decode_lost(CELTDecoder *st, short *pcm)
+static void celt_decode_lost(CELTDecoder *st, short *pcm)
 {
    int i, N, B;
    N = st->block_size;
@@ -397,9 +397,6 @@
    B = st->nb_blocks;
    
    float X[B*N];         /**< Interleaved signal MDCTs */
-   float P[B*N];         /**< Interleaved pitch MDCTs*/
-   float bandE[st->mode->nbEBands];
-   float gains[st->mode->nbPBands];
    int pitch_index;
    
    pitch_index = st->last_pitch_index;
@@ -448,7 +445,7 @@
    ec_dec_init(&dec,&buf);
    
    /* Get the pitch index */
-   pitch_index = ec_dec_uint(&dec, MAX_PERIOD-(B+1)*N);;
+   pitch_index = ec_dec_uint(&dec, MAX_PERIOD-(B+1)*N);
    st->last_pitch_index = pitch_index;
    
    /* Get band energies */
--- a/libcelt/testcelt.c
+++ b/libcelt/testcelt.c
@@ -32,6 +32,7 @@
 
 #include "celt.h"
 #include <stdio.h>
+#include <stdlib.h>
 
 #define FRAME_SIZE 256
 #define NBLOCKS 2
--- a/libcelt/vq.c
+++ b/libcelt/vq.c
@@ -303,7 +303,7 @@
 
 void copy_unquant(float *x, int N, int K, float *Y, int B, int N0, ec_dec *dec)
 {
-   int i,j;
+   int j;
    int sign;
    float s;
    int best;
--- a/libentcode/laplace.c
+++ b/libentcode/laplace.c
@@ -31,7 +31,6 @@
 
 #include "entenc.h"
 #include "entdec.h"
-#include <stdio.h>
 
 static int ec_laplace_get_total(int decay)
 {
@@ -83,7 +82,7 @@
    fl = 0;
    fs = 1<<15;
    fh = fs;
-   while (fm >= fh)
+   while (fm >= fh && fs != 0)
    {
       fl = fh;
       fs = (fs*decay)>>14;
@@ -100,12 +99,15 @@
          fh -= fs;
       }
    }
-   //printf ("fl/fh: %d/%d\n", fl, fh);
+   /* Preventing an infinite loop in case something screws up in the decoding */
+   if (fl==fh)
+      fl--;
    ec_dec_update(dec, fl, fh, ft);
    return val;
 }
 
 #if 0
+#include <stdio.h>
 int main()
 {
    int val;