lvqcl wrote:

> I found that 32-bit total_error_* variables in fixed_compute_best_predictor()
> functions can overflow. For example it happens with the attached test.wav and
> the following options: "-l 0 -b 4200 -m -r 5".
> 
> The attached patch should fix it.
> 
> (Now all three use_wide_by_* variables are unused, so I commented them out
> with #ifdef 0).

Rather than '#ifdef 0'-ing them out, shouldn't we remove them instead?

Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/
_______________________________________________
flac-dev mailing list
flac-dev@xiph.org
http://lists.xiph.org/mailman/listinfo/flac-dev

Reply via email to