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

Uoti Urpala uoti.urpala
Sun Mar 30 07:40:03 CEST 2008


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...

Btw the timestamp of your mail is completely wrong, it's hours into the
future.





More information about the ffmpeg-devel mailing list