[FFmpeg-devel] [PATCH] ALS decoder

Thilo Borgmann thilo.borgmann
Tue Aug 25 19:10:28 CEST 2009


>> +    if (sconf->rlslms) {
>> +        // TODO: read RLSLMS extension data
>> +    }
> 
> Hmmm
Yes, but sconf->rlslms is needed elsewhere. Better to:
// TODO: ...
// if (sconf->...) {
// }
?


> 
>> +            // if joint_stereo and block_switching is set, independent decoding
>> +            // is signaled via the first bit of bs_info
>> +            if(sconf->joint_stereo && sconf->block_switching) {
> 
> if (
> 
>> +    if (sconf->floating) {
>> +        // TODO: read_diff_float_data
>> +    }
> 
> Hmmm
Same as above?


Will be done in rev. 10.


Thanks!

-Thilo



More information about the ffmpeg-devel mailing list