Hi Takehiro.

I still get assertion failures at this point.

Takehiro Tominaga schrieb am Die, 28 M�r 2000:
> Hi, Robert.
> 
> This "cod_info.count1 += 2" is necessary to reduce the code size,
> but we can't reduce the code size when cod_info.count1 == 576.
> 
> this is fixed before you check in the "assert".
> and I think it is OK now...

Sorry, but it is not OK now. If you check count1 == 576 before this,
what happens if count1 == 575? 
575+2=577 --> index overflow --> crash/assertion failure!

So I assume, you have to check count1 >= 575 before.

> ---
> Takehiro TOMINAGA // may the source be with you!
> 
> >>>>> "R" == Robert Hegemann <[EMAIL PROTECTED]> writes:
> 
>     R> I checked the 3.67 tarball from sulaco.org and it seems to
>     R> work.  For the SEGFAULT problem with Takehiro's enhanced code I
>     R> inserted an assert(i<=576) in best_huffman_divide, and now it
>     R> stops with an assertion failure instead of a segmentation
>     R> fault.
> 
> --
> MP3 ENCODER mailing list ( http://geek.rcc.se/mp3encoder/ )

Robert
-- 
http://linux.unixcity.de/catwalk/index.html
--
MP3 ENCODER mailing list ( http://geek.rcc.se/mp3encoder/ )

Reply via email to