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

[Kostya]

On Fri, Jan 15, 2010 at 11:34:43PM +0100, Michael Niedermayer wrote:
> 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
 
since it's used in further calculation in that form, I don't think so
 
> > -                    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 */
> > 
> 
> -- 
> 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.



> _______________________________________________
> FFmpeg-soc mailing list
> FFmpeg-soc at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc