[FFmpeg-devel] [PATCH] ALS decoder

Diego Biurrun diego
Tue Aug 25 19:35:32 CEST 2009


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.

Diego



More information about the ffmpeg-devel mailing list