On Fri, Nov 28, 2014 at 2:50 PM, Luca Barbato <[email protected]> wrote:
> On 25/11/14 02:54, Vittorio Giovara wrote:
>>
>> -    int i, value, exponent;
>> +    int i, exponent;
>>       for (i = 1; i < TABLE_4_3_SIZE; i++) {
>> -        double value = i / 4;
>> +        int value = i / 4;
>>           double f, fm;
>>           int e, m;
>>           /* cbrtf() isn't available on all systems, so we use powf(). */
>> -        f  = value * powf(value, 1.0 / 3.0) * pow(2, (i & 3) * 0.25);
>> +        f  = powf(value, 1.0 / 3.0) * pow(2, (i & 3) * 0.25) * value;
>>           fm = frexp(f, &e);
>>           m  = (uint32_t)(fm * (1LL << 31) + 0.5);
>>           e += FRAC_BITS - 31 + 5 - 100;
>> @@ -58,6 +58,7 @@ static void mpegaudio_tableinit(void)
>>           table_4_3_exp[i]   = -e;
>>       }
>>       for (exponent = 0; exponent < 512; exponent++) {
>> +        int value;
>
>
> Why not keep value outside the loop?

sure ok, i'll resend it.
Any idea if I should change the order of operands?
-- 
Vittorio
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to