[FFmpeg-devel] [PATCH] added expr evaluation to drawtext - fontsize

Michael Niedermayer michael at niedermayer.cc
Thu Sep 15 13:20:12 EEST 2016


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160915/c95c08e2/attachment.sig>


More information about the ffmpeg-devel mailing list