[FFmpeg-devel] [PATCH] IFF demuxer and 8SVX decoder

Jai Menon realityman
Sun Mar 30 13:26:16 CEST 2008


On Sunday 30 March 2008 05:40:03 Uoti Urpala wrote:
> On Sun, 2008-03-30 at 10:43 +0000, Jai Menon wrote:
> > On Saturday 29 March 2008 14:46:01 Michael Niedermayer wrote:
> > > Currently the code does practically:
> > >
> > > while(not end){
> > >     acc += constant_table[*x++];
> > >     *out++= acc<<8;
> > > }
> > >
> > > The code can be changed so it does not need the "<<8" while keeping the
> > > output the same 16bit data.
> >
> > I have removed the shifts. The changes should clarify some stuff on why
> > the previous code was like that.
>
> But you added a bswap in the loop instead of changing the table. I don't
> think that is what was intended...
 
exactly....neither do i !!! i just wanted to write the decoding step in a 
different way to show why i prefer the previous method. But i'm still 
experimenting .......
> Btw the timestamp of your mail is completely wrong, it's hours into the
> future.

 Could be my timezone ? any ideas on how to "fix" this...........




More information about the ffmpeg-devel mailing list