[FFmpeg-devel] [PATCH] dca: fix dca_bitalloc_index decoder init

Michael Niedermayer michaelni
Wed May 23 20:25:31 CEST 2007


Hi

On Wed, May 23, 2007 at 10:09:01PM +0400, Sergey Vlasov wrote:
> 
> dca: fix dca_bitalloc_index decoder init
> 
> One of Huffman codes used in dca_bitalloc_index has 10-bit codes, but
> the decode table is built only for 9 bits, therefore the .wrap field
> must be set to 2 for correct decoding.

patch ok, and as you are already working with that code, maybe you
want to replace all the wraps by (named) constants, or just replace them all
by a literal 2 both should make the code faster, reading the wrap out of a
struct completely defeats the purpose (which is to skip some check)

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070523/7c27faa4/attachment.pgp>



More information about the ffmpeg-devel mailing list