[FFmpeg-devel] [PATCH] ALS decoder

[Thilo Borgmann]

Diego Biurrun schrieb:
> On Tue, Aug 25, 2009 at 07:10:28PM +0200, Thilo Borgmann wrote:
>>>> +    if (sconf->rlslms) {
>>>> +        // TODO: read RLSLMS extension data
>>>> +    }
>>> Hmmm
>> Yes, but sconf->rlslms is needed elsewhere. Better to:
>> // TODO: ...
>> // if (sconf->...) {
>> // }
>> ?
> 
> I don't like accumulating cruft.  I would suggest just some TODO
> comments in-place or at the top of the file.
Ok.

-Thilo