On Wed, Jul 17, 2013 at 07:45:53PM +1000, Erik de Castro Lopo wrote: > The fix was changing one local variable from FLAC_uint32 to FLAC_uint64 > in function precompute_partition_info_sums_(). > > > https://git.xiph.org/?p=flac.git;a=commit;h=6f7ec60c7e7f05f5ab0b1cf6b7b0945e44afcd4b
I don't like this fix. It will probably hurt performance with 16-bit data and it won't fix the problem in the assembly code. I think the check if 32-bit accumulator is enough should be improved instead if possible. -- Miroslav Lichvar _______________________________________________ flac-dev mailing list flac-dev@xiph.org http://lists.xiph.org/mailman/listinfo/flac-dev
