[FFmpeg-devel] [PATCH] Implement AAC Long Term Prediction (LTP) decoding module

Young Han Lee cpumaker
Wed Feb 2 19:25:56 CET 2011


On Tue, Feb 1, 2011 at 10:44 PM, Diego Biurrun <diego at biurrun.de> wrote:

> On Tue, Feb 01, 2011 at 12:50:42PM +0000, M?ns Rullg?rd wrote:
> > Diego Biurrun <diego at biurrun.de> writes:
> >
> > > On Tue, Feb 01, 2011 at 10:57:28AM +0900, Young Han Lee wrote:
> > >>
> > >> please check it again. :)
> > >
> > > Looks good now.  Some more nits for your Doxygen comments below, but
> > > don't bother sending a new patch.  Wait for the review from Alex first.
> > >
> > >> --- a/libavcodec/aacdec.c
> > >> +++ b/libavcodec/aacdec.c
> > >> @@ -1682,23 +1706,108 @@ static void apply_tns(float coef[1024],
> TemporalNoiseShaping *tns,
> > >> +/**
> > >> + * Windowing and MDCT to obtain the spectral coefficient from the
> predicted sample by LTP
> > >
> > > This is a longish line that you could break and end in a period.
> > > The sentence would also be easier to understand if there was a
> > > verb in it.
> >
> > "Obtain" is a verb.  Agree about the line length.
>
> "Obtain", however, is not the predicate of the sentence.  With predicate
> it should read something like
>
>  Apply/use/frobnicate windowing and MDCT to obtain the spectral
>  coefficient from the predicted sample by LTP.
>
> I got it. I'll apply the comment with Alex's review.

 Diego
>
>
Young Han

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel at mplayerhq.hu
https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel



More information about the ffmpeg-devel mailing list