aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAnton Khirnov <anton@khirnov.net>2024-10-12 18:26:50 +0200
committerAnton Khirnov <anton@khirnov.net>2024-10-15 10:39:14 +0200
commit92981320b37417e24232f00ece742481b788a2d1 (patch)
tree9ec3bd88fc43feaf38f55fe14b79c2924538fa0c
parent8160178dfc0e6bdaacf80dec58e595a9d595eedc (diff)
downloadffmpeg-92981320b37417e24232f00ece742481b788a2d1.tar.gz
lavfi/vf_libplacebo: convert to query_func2()
Drop a redundant error message for out_format, as it is already validated in init.
-rw-r--r--libavfilter/vf_libplacebo.c24
1 files changed, 11 insertions, 13 deletions
diff --git a/libavfilter/vf_libplacebo.c b/libavfilter/vf_libplacebo.c
index 5bf8b7e77a..77219b830a 100644
--- a/libavfilter/vf_libplacebo.c
+++ b/libavfilter/vf_libplacebo.c
@@ -1090,10 +1090,12 @@ static int libplacebo_activate(AVFilterContext *ctx)
return FFERROR_NOT_READY;
}
-static int libplacebo_query_format(AVFilterContext *ctx)
+static int libplacebo_query_format(const AVFilterContext *ctx,
+ AVFilterFormatsConfig **cfg_in,
+ AVFilterFormatsConfig **cfg_out)
{
int err;
- LibplaceboContext *s = ctx->priv;
+ const LibplaceboContext *s = ctx->priv;
const AVPixFmtDescriptor *desc = NULL;
AVFilterFormats *infmts = NULL, *outfmts = NULL;
@@ -1139,29 +1141,25 @@ static int libplacebo_query_format(AVFilterContext *ctx)
}
if (!infmts || !outfmts) {
- if (s->out_format) {
- av_log(s, AV_LOG_ERROR, "Invalid output format '%s'!\n",
- av_get_pix_fmt_name(s->out_format));
- }
err = AVERROR(EINVAL);
goto fail;
}
for (int i = 0; i < s->nb_inputs; i++)
- RET(ff_formats_ref(infmts, &ctx->inputs[i]->outcfg.formats));
- RET(ff_formats_ref(outfmts, &ctx->outputs[0]->incfg.formats));
+ RET(ff_formats_ref(infmts, &cfg_in[i]->formats));
+ RET(ff_formats_ref(outfmts, &cfg_out[0]->formats));
/* Set colorspace properties */
- RET(ff_formats_ref(ff_all_color_spaces(), &ctx->inputs[0]->outcfg.color_spaces));
- RET(ff_formats_ref(ff_all_color_ranges(), &ctx->inputs[0]->outcfg.color_ranges));
+ RET(ff_formats_ref(ff_all_color_spaces(), &cfg_in[0]->color_spaces));
+ RET(ff_formats_ref(ff_all_color_ranges(), &cfg_in[0]->color_ranges));
outfmts = s->colorspace > 0 ? ff_make_formats_list_singleton(s->colorspace)
: ff_all_color_spaces();
- RET(ff_formats_ref(outfmts, &ctx->outputs[0]->incfg.color_spaces));
+ RET(ff_formats_ref(outfmts, &cfg_out[0]->color_spaces));
outfmts = s->color_range > 0 ? ff_make_formats_list_singleton(s->color_range)
: ff_all_color_ranges();
- RET(ff_formats_ref(outfmts, &ctx->outputs[0]->incfg.color_ranges));
+ RET(ff_formats_ref(outfmts, &cfg_out[0]->color_ranges));
return 0;
fail:
@@ -1464,7 +1462,7 @@ const AVFilter ff_vf_libplacebo = {
.activate = &libplacebo_activate,
.process_command = &libplacebo_process_command,
FILTER_OUTPUTS(libplacebo_outputs),
- FILTER_QUERY_FUNC(libplacebo_query_format),
+ FILTER_QUERY_FUNC2(libplacebo_query_format),
.priv_class = &libplacebo_class,
.flags_internal = FF_FILTER_FLAG_HWFRAME_AWARE,
.flags = AVFILTER_FLAG_HWDEVICE | AVFILTER_FLAG_DYNAMIC_INPUTS,