[FFmpeg-soc] [soc]: r5127 - als/alsdec.c

Thilo Borgmann thilo.borgmann at googlemail.com
Sat Aug 15 15:01:22 CEST 2009


>> +                unsigned int max, convert;
>>  ...
>> +                if (smp < progressive) {
>> +                    max     = smp;
>> +                    convert = 1;
>> +                } else {
>> +                    max     = progressive;
>> +                    convert = 0;
>> +                }

> 
> Hmm...
> tried
> max = smp;
> convert = 1;
> outside the loop and inside
> if (smp == progessive) {
> max = progressive;
> convert = 0;
> }
> ?

smp is the loop counter and can therefore not be put outside the loop.
For the convert assignment, this could have been true but since there
have to be two if's (if smp==progressive  and if smp < progressive), it
has shown to be even slower than the two loop version.

Sorry and thanks for reviewing :)

Thilo


More information about the FFmpeg-soc mailing list