On Fri, Sep 02, 2016 at 14:10:41 +0200, Moritz Barsnick wrote: > > + 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.)
Sorry, probably makes sense, equal to if ((ret = update_fontsize(ctx)) != 0) { I'm still not *really* sure whether the second set of brackets is required to have "if" check the assigned ret (I thought not). I'll let someone else judge. Moritz _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel