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

Jai Menon realityman
Sun Mar 30 13:43:32 CEST 2008


On Sunday 30 March 2008 11:26:16 Jai Menon wrote:
> 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 .......

Another interpretation........

Regards
Jai Menon
<realityman at gmx.net>


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 8svx_decoder.patch
Type: text/x-diff
Size: 5251 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080330/c56144c9/attachment.patch>



More information about the ffmpeg-devel mailing list