[FFmpeg-devel] [PATCH] NellyMoser audio decoder v2
Michael Niedermayer
michaelni
Mon Oct 8 03:55:23 CEST 2007
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>
More information about the ffmpeg-devel
mailing list