[FFmpeg-cvslog] r9135 - in trunk/tests: Makefile tiny_psnr.c

[Diego Biurrun]

On Sun, May 27, 2007 at 05:14:36PM +0200, Diego Biurrun wrote:
> On Sun, May 27, 2007 at 03:36:02PM +0200, diego wrote:
> > 
> > Log:
> > Fix seek_test compilation for out-of-tree builds.
> > 
> > Modified:
> >    trunk/tests/Makefile
> >    trunk/tests/tiny_psnr.c
> > 
> > --- trunk/tests/tiny_psnr.c	(original)
> > +++ trunk/tests/tiny_psnr.c	Sun May 27 15:36:02 2007
> > @@ -49,19 +49,7 @@ uint64_t exp16_table[21]={
> >   195360063,
> >   582360139072LL,
> >  };
> > -#if 1
> > -// 16.16 fixpoint exp()
> > -static unsigned int exp16(unsigned int a){
> > -    int i;
> > -    int out= 1<<16;
> >  
> > -    for(i=19;i>=0;i--){
> > -        if(a&(1<<i))
> > -            out= (out*exp16_table[i] + (1<<15))>>16;
> > -    }
> > -
> > -    return out;
> > -}
> >  // 16.16 fixpoint log()
> >  static int64_t log16(uint64_t a){
> >      int i;
> > @@ -80,7 +68,6 @@ static int64_t log16(uint64_t a){
> >      return out;
> >  }
> >  
> > -#endif
> >  static uint64_t int_sqrt(uint64_t a)
> >  {
> >      uint64_t ret=0;
> 
> Ouch!
> 
> Dunno, how this slipped through, I could swear that I looked at the diff
> before committing as I always do...

Reverted.

While we're at it, is it OK to remove the '#if 1' and/or the unused
function?

Diego