[FFmpeg-devel] [PATCH] NellyMoser audio decoder v2

[Michael Niedermayer]

Hi

On Sun, Oct 07, 2007 at 08:16:05PM +0200, Benjamin Larsson wrote:
> Latest version of the patch. Is it ok to commit now ?

what happened with the points where you replied "ill look into it"?



[...]
> +    if (bitsum != NELLY_DETAIL_BITS) {
> +        shift = 0;
> +        diff = bitsum - NELLY_DETAIL_BITS;
> +
> +        while (FFABS(diff) <= 16383) {
> +            shift++;
> +            diff *= 2;
> +        }
> +
> +        diff = (diff * NELLY_BASE_OFF) >> 15;
> +        shift = shift_saved-(NELLY_BASE_SHIFT+shift-15);
> +
> +        diff = signed_shift(diff, shift);
> +
> +        for (j = 1; j < 20; j++) {
> +            tmp = off;
> +            off += diff;
> +            last_bitsum = bitsum;
> +
> +            bitsum = sum_bits(sbuf, shift_saved, off);
> +
> +            if ((bitsum-NELLY_DETAIL_BITS) * (last_bitsum-NELLY_DETAIL_BITS) <= 0)
> +                break;
> +        }
> +
> +        if (bitsum != NELLY_DETAIL_BITS) {
> +            if (bitsum > NELLY_DETAIL_BITS) {
> +                big_off = off;
> +                off = tmp;
> +                big_bitsum=bitsum;
> +                small_bitsum=last_bitsum;
> +            } else {
> +                big_off = tmp;
> +                big_bitsum=last_bitsum;
> +                small_bitsum=bitsum;
> +            }
> +
> +            while (bitsum != NELLY_DETAIL_BITS && j <= 19) {
> +                diff = (big_off+off)>>1;

between what and what is this a difference?

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071008/41c07bc7/attachment.pgp>