From 6345184fc9390ea6fe61b11e3aae3a716c2dff45 Mon Sep 17 00:00:00 2001
From: Mashiat Sarker Shakkhar <shahriman_ams@yahoo.com>
Date: Sun, 22 Jan 2012 17:48:15 +0600
Subject: [PATCH] Support 24-bit decoding

---
 libavcodec/wmalosslessdec.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/libavcodec/wmalosslessdec.c b/libavcodec/wmalosslessdec.c
index 6acfad7815..f8fcf363a2 100644
--- a/libavcodec/wmalosslessdec.c
+++ b/libavcodec/wmalosslessdec.c
@@ -240,7 +240,7 @@ typedef struct WmallDecodeCtx {
     int8_t mclms_scaling;
     int16_t mclms_coeffs[128];
     int16_t mclms_coeffs_cur[4];
-    int16_t mclms_prevvalues[64];   // FIXME: should be 32-bit / 16-bit depending on bit-depth
+    int mclms_prevvalues[64];   // FIXME: should be 32-bit / 16-bit depending on bit-depth
     int16_t mclms_updates[64];
     int mclms_recent;
 
@@ -253,7 +253,7 @@ typedef struct WmallDecodeCtx {
 	int coefsend;
 	int bitsend;
 	int16_t coefs[256];
-    int16_t lms_prevvalues[512];    // FIXME: see above
+    int lms_prevvalues[512];    // FIXME: see above
     int16_t lms_updates[512];   // and here too
     int recent;
     } cdlms[2][9];              /* XXX: Here, 2 is the max. no. of channels allowed,
@@ -770,7 +770,7 @@ static void clear_codec_buffers(WmallDecodeCtx *s)
     for (ich = 0; ich < s->num_channels; ich++) {
         for (ilms = 0; ilms < s->cdlms_ttl[ich]; ilms++) {
             memset(s->cdlms[ich][ilms].coefs         , 0, 256 * sizeof(int16_t));
-            memset(s->cdlms[ich][ilms].lms_prevvalues, 0, 512 * sizeof(int16_t));
+            memset(s->cdlms[ich][ilms].lms_prevvalues, 0, 512 * sizeof(int));
             memset(s->cdlms[ich][ilms].lms_updates   , 0, 512 * sizeof(int16_t));
         }
         s->ave_sum[ich] = 0;
@@ -804,7 +804,7 @@ static void mclms_update(WmallDecodeCtx *s, int icoef, int *pred)
     int order = s->mclms_order;
     int num_channels = s->num_channels;
     int range = 1 << (s->bits_per_sample - 1);
-    int bps = s->bits_per_sample > 16 ? 4 : 2; // bytes per sample
+    //int bps = s->bits_per_sample > 16 ? 4 : 2; // bytes per sample
 
     for (ich = 0; ich < num_channels; ich++) {
         pred_error = s->channel_residues[ich][icoef] - pred[ich];
@@ -849,10 +849,10 @@ static void mclms_update(WmallDecodeCtx *s, int icoef, int *pred)
     if (s->mclms_recent == 0) {
         memcpy(&s->mclms_prevvalues[order * num_channels],
                s->mclms_prevvalues,
-               bps * order * num_channels);
+               4 * order * num_channels);
         memcpy(&s->mclms_updates[order * num_channels],
                s->mclms_updates,
-               bps * order * num_channels);
+               2 * order * num_channels);
         s->mclms_recent = num_channels * order;
     }
 }
@@ -911,7 +911,7 @@ static void lms_update(WmallDecodeCtx *s, int ich, int ilms, int input, int resi
     int icoef;
     int recent = s->cdlms[ich][ilms].recent;
     int range = 1 << s->bits_per_sample - 1;
-    int bps = s->bits_per_sample > 16 ? 4 : 2; // bytes per sample
+    //int bps = s->bits_per_sample > 16 ? 4 : 2; // bytes per sample
 
     if (residue < 0) {
         for (icoef = 0; icoef < s->cdlms[ich][ilms].order; icoef++)
@@ -931,10 +931,10 @@ static void lms_update(WmallDecodeCtx *s, int ich, int ilms, int input, int resi
                 follow kshishkov's suggestion of using a union. */
         memcpy(&s->cdlms[ich][ilms].lms_prevvalues[s->cdlms[ich][ilms].order],
                s->cdlms[ich][ilms].lms_prevvalues,
-               bps * s->cdlms[ich][ilms].order);
+               4 * s->cdlms[ich][ilms].order);
         memcpy(&s->cdlms[ich][ilms].lms_updates[s->cdlms[ich][ilms].order],
                s->cdlms[ich][ilms].lms_updates,
-               bps * s->cdlms[ich][ilms].order);
+               2 * s->cdlms[ich][ilms].order);
         recent = s->cdlms[ich][ilms].order - 1;
     }