[FFmpeg-cvslog] r14873 - in trunk: Changelog doc/general.texi libavcodec/Makefile libavcodec/aac.c libavcodec/allcodecs.c libavcodec/avcodec.h

Diego Biurrun diego
Thu Aug 21 12:31:14 CEST 2008


On Thu, Aug 21, 2008 at 11:24:28AM +0100, Robert Swain wrote:
> 2008/8/21 Diego Biurrun <diego at biurrun.de>:
> > On Thu, Aug 21, 2008 at 09:21:27AM +0200, superdump wrote:
> >>
> >> Log:
> >> Last hunk of the AAC decoder code to be OKed and build system and documentation
> >> alterations as appropriate
> >
> > And much rejoicing was heard throughout the land \o/
> >
> >> --- trunk/libavcodec/aac.c    (original)
> >> +++ trunk/libavcodec/aac.c    Thu Aug 21 09:21:26 2008
> >> @@ -1104,6 +1104,42 @@ static int decode_extension_payload(AACC
> >> +            /* tns_decode_coef
> >> +             * FIXME: This duplicates the functionality of some double code in lpc.c.
> >> +             */
> >> +            for (m = 0; m < order; m++) {
> >> +               float tmp;
> >> +               lpc[m] = tns->coef[w][filt][m];
> >
> > Incorrect indentation; this is missing one space.
> 
> Oops. Copy and paste error. The code in lpc.c uses 3 space indentation
> I believe.

Shame on the committer.  Fixed.

Diego




More information about the ffmpeg-cvslog mailing list