diff mbox

[FFmpeg-devel,PATCHv2] avfiltergraph.c: restore disabling of auto conversions

Message ID 1470820188-31893-1-git-send-email-pburt0@gmail.com
State Accepted
Commit 16ea0bca14730d71dd319b7dc148598275928cfe
Headers show

Commit Message

Burt P Aug. 10, 2016, 9:09 a.m. UTC
Restore a check added in 440af105f2306d3c7b3b3f4d7530bab910d49cb9
but lost sometime after. avfilter_graph_set_auto_convert() will
have an effect once again.

Signed-off-by: Burt P <pburt0@gmail.com>
---
 libavfilter/avfiltergraph.c | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Burt P Aug. 10, 2016, 4:44 p.m. UTC | #1
applied

On Wed, Aug 10, 2016 at 4:09 AM, Burt P <pburt0@gmail.com> wrote:
> Restore a check added in 440af105f2306d3c7b3b3f4d7530bab910d49cb9
> but lost sometime after. avfilter_graph_set_auto_convert() will
> have an effect once again.
>
> Signed-off-by: Burt P <pburt0@gmail.com>
> ---
>  libavfilter/avfiltergraph.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/libavfilter/avfiltergraph.c b/libavfilter/avfiltergraph.c
> index 4275113..3af698d 100644
> --- a/libavfilter/avfiltergraph.c
> +++ b/libavfilter/avfiltergraph.c
> @@ -507,6 +507,14 @@ static int query_formats(AVFilterGraph *graph, AVClass *log_ctx)
>                  char scale_args[256];
>                  char inst_name[30];
>
> +                if (graph->disable_auto_convert) {
> +                    av_log(log_ctx, AV_LOG_ERROR,
> +                           "The filters '%s' and '%s' do not have a common format "
> +                           "and automatic conversion is disabled.\n",
> +                           link->src->name, link->dst->name);
> +                    return AVERROR(EINVAL);
> +                }
> +
>                  /* couldn't merge format lists. auto-insert conversion filter */
>                  switch (link->type) {
>                  case AVMEDIA_TYPE_VIDEO:
> --
> 2.7.4
>
diff mbox

Patch

diff --git a/libavfilter/avfiltergraph.c b/libavfilter/avfiltergraph.c
index 4275113..3af698d 100644
--- a/libavfilter/avfiltergraph.c
+++ b/libavfilter/avfiltergraph.c
@@ -507,6 +507,14 @@  static int query_formats(AVFilterGraph *graph, AVClass *log_ctx)
                 char scale_args[256];
                 char inst_name[30];
 
+                if (graph->disable_auto_convert) {
+                    av_log(log_ctx, AV_LOG_ERROR,
+                           "The filters '%s' and '%s' do not have a common format "
+                           "and automatic conversion is disabled.\n",
+                           link->src->name, link->dst->name);
+                    return AVERROR(EINVAL);
+                }
+
                 /* couldn't merge format lists. auto-insert conversion filter */
                 switch (link->type) {
                 case AVMEDIA_TYPE_VIDEO: