On Fri, Sep 09, 2016 at 05:26:03PM -0700, Brett Harrison wrote:
> Here are the changes requested
[...]
> +static av_cold int parse_fontsize(AVFilterContext *ctx)
> +{
> + DrawTextContext *s = ctx->priv;
> + int err;
> +
> + if (s->fontsize_expr == NULL)
> + return AVERROR(EINVAL);
> +
> + av_expr_free(s->fontsize_pexpr);
> + s->fontsize_pexpr = NULL;
> +
> + if ((err = av_expr_parse(&s->fontsize_pexpr, s->fontsize_expr, var_names,
> + NULL, NULL, fun2_names, fun2, 0, ctx)) < 0)
> + return err;
> +
> + return 0;
> +}why is av_expr_parse() not executed where the other av_expr_parse() are ? [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Freedom in capitalist society always remains about the same as it was in ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
