summaryrefslogtreecommitdiffstats
path: root/audio/filter/af_lavrresample.c
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2015-07-19 22:50:08 +0200
committerwm4 <wm4@nowhere>2015-07-19 22:50:08 +0200
commit8749900b5fa6f1766e7ddf2e836d2417a034eff2 (patch)
treed3be91658b55fedaf5d9920ac4e796ee750a7fb4 /audio/filter/af_lavrresample.c
parent2cf019ed707391759828cf1d4e81f1b235d95bc7 (diff)
downloadmpv-8749900b5fa6f1766e7ddf2e836d2417a034eff2.tar.bz2
mpv-8749900b5fa6f1766e7ddf2e836d2417a034eff2.tar.xz
af_lavrresample: don't unnecessarily print remix message
This message bloats verbose log output if e.g. audio speed is frequently readjusted, such as when syncing audio to video. So don't print the message if only speed is changed. (This case requires reconfiguration, but can't change the input/output channel maps.) Also do not print the message if no remixing is done at all.
Diffstat (limited to 'audio/filter/af_lavrresample.c')
-rw-r--r--audio/filter/af_lavrresample.c16
1 files changed, 8 insertions, 8 deletions
diff --git a/audio/filter/af_lavrresample.c b/audio/filter/af_lavrresample.c
index a103c46dd9..e2c5ff95e1 100644
--- a/audio/filter/af_lavrresample.c
+++ b/audio/filter/af_lavrresample.c
@@ -212,7 +212,7 @@ static void transpose_order(int *map, int num)
}
static int configure_lavrr(struct af_instance *af, struct mp_audio *in,
- struct mp_audio *out)
+ struct mp_audio *out, bool verbose)
{
struct af_resample *s = af->priv;
@@ -274,8 +274,10 @@ static int configure_lavrr(struct af_instance *af, struct mp_audio *in,
mp_chmap_from_lavc(&in_lavc, in_ch_layout);
mp_chmap_from_lavc(&out_lavc, out_ch_layout);
- MP_VERBOSE(af, "Remix: %s -> %s\n", mp_chmap_to_str(&in_lavc),
- mp_chmap_to_str(&out_lavc));
+ if (verbose && !mp_chmap_equals(&in_lavc, &out_lavc)) {
+ MP_VERBOSE(af, "Remix: %s -> %s\n", mp_chmap_to_str(&in_lavc),
+ mp_chmap_to_str(&out_lavc));
+ }
if (in_lavc.num != map_in.num) {
// For handling NA channels, we would have to add a planarization step.
@@ -339,9 +341,7 @@ static int configure_lavrr(struct af_instance *af, struct mp_audio *in,
// * Also, the input channel layout must have already been set.
avresample_set_channel_mapping(s->avrctx, s->reorder_in);
- if (avresample_open(s->avrctx) < 0 ||
- avresample_open(s->avrctx_out) < 0)
- {
+ if (avresample_open(s->avrctx) < 0 || avresample_open(s->avrctx_out) < 0) {
MP_ERR(af, "Cannot open Libavresample Context. \n");
goto error;
}
@@ -385,7 +385,7 @@ static int control(struct af_instance *af, int cmd, void *arg)
? AF_OK : AF_FALSE;
if (r == AF_OK && needs_lavrctx_reconfigure(s, in, out))
- r = configure_lavrr(af, in, out);
+ r = configure_lavrr(af, in, out, true);
return r;
}
case AF_CONTROL_SET_FORMAT: {
@@ -411,7 +411,7 @@ static int control(struct af_instance *af, int cmd, void *arg)
// in the resampler.
af->filter_frame(af, NULL);
// Reinitialize resampler.
- configure_lavrr(af, &af->fmt_in, &af->fmt_out);
+ configure_lavrr(af, &af->fmt_in, &af->fmt_out, false);
}
return AF_OK;
}