[FFmpeg-devel] [PATCH] Fix for inverted sign in ffvorbis audio?output (resubmit)

Diego Biurrun diego
Tue May 12 09:29:10 CEST 2009


On Tue, May 12, 2009 at 10:18:57AM +0300, Siarhei Siamashka wrote:
> On Sunday 10 May 2009, Reimar D?ffinger wrote:
> > On Sun, May 10, 2009 at 03:36:13PM +0300, Siarhei Siamashka wrote:
> > But what I meant is that in
> >
> > > -    for(i=0;i<n4;i++) {
> > > -        alpha = 2 * M_PI * (i + 1.0 / 8.0) / n;
> > > -        s->tcos[i] = -cos(alpha);
> > > -        s->tsin[i] = -sin(alpha);
> > > +    if (scale < 0.0) {
> > > +        scale = sqrt(-scale);
> > > +        for(i=0;i<n4;i++) {
> > > +            alpha = 2 * M_PI * (i + 1.0 / 8.0) / n;
> > > +            s->tcos[i] = sin(alpha) * scale;
> > > +            s->tsin[i] = -cos(alpha) * scale;
> > > +        }
> > > +    } else {
> > > +        scale = sqrt(scale);
> > > +        for(i=0;i<n4;i++) {
> > > +            alpha = 2 * M_PI * (i + 1.0 / 8.0) / n;
> > > +            s->tcos[i] = -cos(alpha) * scale;
> > > +            s->tsin[i] = -sin(alpha) * scale;
> > > +        }
> >
> > Reindenting the "for ..." and "alpha = ..." is cosmetics.
> 
> FFmpeg documentation does not quite agree with you (you are too extreme in
> this case):

That's MPlayer documentation.

Just reindent in a separate commit.

Diego



More information about the ffmpeg-devel mailing list