diff mbox

[FFmpeg-devel] avfilter/af_atempo: Make independent of the channel layout

Message ID 20160905223747.17623-1-michael@niedermayer.cc
State Accepted
Commit db3b93319dc806fa437215c9224d945e81083593
Headers show

Commit Message

Michael Niedermayer Sept. 5, 2016, 10:37 p.m. UTC
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 libavfilter/af_atempo.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Paul B Mahol Sept. 5, 2016, 10:59 p.m. UTC | #1
On 9/6/16, Michael Niedermayer <michael@niedermayer.cc> wrote:
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  libavfilter/af_atempo.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>

lgtm
Pavel Koshevoy Sept. 6, 2016, 4:25 a.m. UTC | #2
On Sep 5, 2016 4:41 PM, "Michael Niedermayer" <michael@niedermayer.cc>
wrote:
>
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  libavfilter/af_atempo.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/libavfilter/af_atempo.c b/libavfilter/af_atempo.c
> index 7b3d57c..d5d29b3 100644
> --- a/libavfilter/af_atempo.c
> +++ b/libavfilter/af_atempo.c
> @@ -1044,9 +1044,8 @@ static int config_props(AVFilterLink *inlink)
>
>      enum AVSampleFormat format = inlink->format;
>      int sample_rate = (int)inlink->sample_rate;
> -    int channels = av_get_channel_layout_nb_channels(inlink->channel_
layout);
>
> -    return yae_reset(atempo, format, sample_rate, channels);
> +    return yae_reset(atempo, format, sample_rate, inlink->channels);
>  }
>
>  static int push_samples(ATempoContext *atempo,
> --
> 2.9.3


Looks good to me,

    Pavel.
Michael Niedermayer Sept. 6, 2016, 10:14 a.m. UTC | #3
On Mon, Sep 05, 2016 at 10:25:47PM -0600, Pavel Koshevoy wrote:
> On Sep 5, 2016 4:41 PM, "Michael Niedermayer" <michael@niedermayer.cc>
> wrote:
> >
> > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> > ---
> >  libavfilter/af_atempo.c | 3 +--
> >  1 file changed, 1 insertion(+), 2 deletions(-)
> >
> > diff --git a/libavfilter/af_atempo.c b/libavfilter/af_atempo.c
> > index 7b3d57c..d5d29b3 100644
> > --- a/libavfilter/af_atempo.c
> > +++ b/libavfilter/af_atempo.c
> > @@ -1044,9 +1044,8 @@ static int config_props(AVFilterLink *inlink)
> >
> >      enum AVSampleFormat format = inlink->format;
> >      int sample_rate = (int)inlink->sample_rate;
> > -    int channels = av_get_channel_layout_nb_channels(inlink->channel_
> layout);
> >
> > -    return yae_reset(atempo, format, sample_rate, channels);
> > +    return yae_reset(atempo, format, sample_rate, inlink->channels);
> >  }
> >
> >  static int push_samples(ATempoContext *atempo,
> > --
> > 2.9.3
> 
> 
> Looks good to me,

applied

thx to all

[...]
diff mbox

Patch

diff --git a/libavfilter/af_atempo.c b/libavfilter/af_atempo.c
index 7b3d57c..d5d29b3 100644
--- a/libavfilter/af_atempo.c
+++ b/libavfilter/af_atempo.c
@@ -1044,9 +1044,8 @@  static int config_props(AVFilterLink *inlink)
 
     enum AVSampleFormat format = inlink->format;
     int sample_rate = (int)inlink->sample_rate;
-    int channels = av_get_channel_layout_nb_channels(inlink->channel_layout);
 
-    return yae_reset(atempo, format, sample_rate, channels);
+    return yae_reset(atempo, format, sample_rate, inlink->channels);
 }
 
 static int push_samples(ATempoContext *atempo,