[FFmpeg-soc] [soc]: r5587 - indeo5/ivi_common.c

[Michael Niedermayer]

On Fri, Jan 15, 2010 at 10:30:31AM +0100, kostya wrote:
> Author: kostya
> Date: Fri Jan 15 10:30:31 2010
> New Revision: 5587
> 
> Log:
> refactor dequantization a bit
> 
> Modified:
>    indeo5/ivi_common.c
> 
> Modified: indeo5/ivi_common.c
> ==============================================================================
> --- indeo5/ivi_common.c	Fri Jan 15 10:19:35 2010	(r5586)
> +++ indeo5/ivi_common.c	Fri Jan 15 10:30:31 2010	(r5587)
> @@ -383,11 +383,8 @@ int ff_ivi_decode_blocks(GetBitContext *
>  
>                      q = (base_tab[pos] * scale_tab[quant]) >> 8;

>                      q += !q; // ensure the q always >= 1

looks useless after this commit


> -                    if (q != 1) {
> -                        if (val > 0) {
> -                            val = (val * q) + (q >> 1) - (q & 1);
> -                        } else
> -                            val = (val * q) - (q >> 1) + (q & 1);
> +                    if (q > 1) {
> +                        val = val * q + FFSIGN(val) * ((q >> 1) - (q & 1));
>                      }
>                      trvec[pos] = val;
>                      col_flags[pos & col_mask] |= val; /* track columns containing non-zero coeffs */
> _______________________________________________
> FFmpeg-soc mailing list
> FFmpeg-soc at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc
> 

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Old school: Use the lowest level language in which you can solve the problem
            conveniently.
New school: Use the highest level language in which the latest supercomputer
            can solve the problem without the user falling asleep waiting.
-------------- 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-soc/attachments/20100115/6869cc92/attachment.pgp>