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

[Moritz Barsnick]

On Thu, Sep 01, 2016 at 18:17:11 -0700, Brett Harrison wrote:
> Most recent patch.  I was evaluating fontsize too early before when using
> the 'n' variable in equations.

> +        av_log(ctx, AV_LOG_ERROR, "Font not open\n");

I was wondering: Was does this message tell the user?

> +    if ((ret = update_fontsize(ctx))) {
You were meaning to write
       if (ret = update_fontsize(ctx)) {
or
       if ((ret = update_fontsize(ctx)) < 0) {
?? (Too many brackets the way you did it.)

Moritz