[Ffmpeg-devel] [PATCH] Vorbis I floor type 0 support
Oded Shimon
ods15
Fri Feb 3 08:44:48 CET 2006
On Fri, Jan 27, 2006 at 06:14:35PM +0100, Alexander Strasser wrote:
> Alexander Strasser wrote:
> > Benjamin Larsson wrote:
> [...]
> > > Indentation. It's some more also.
> >
> > Right, there were some more remaining style errors than i thought.
> > ( This happens when you look at your own code too many times ^^" )
> > Hopefully i caught all of them now; new patch attached.
> >
>
> Sorry, really attached now.
[...]
> + vorbis_floor_decode_func floor_decode;
You used a single a variable for pointer to decode function. There's no
garuntee anywhere in the Vorbis spec that all floors in a single file are
the same type. You should make this variable in the floor sturct, and per
mapping/floor.
Other than that, I like this patch... Only thing missing making FFmpeg
Vorbis decoder complete...
- ods15
More information about the ffmpeg-devel
mailing list