[FFmpeg-devel] [PATCH] delogo filter: new "uglarm" interpolation mode added
Carl Eugen Hoyos
ceffmpeg at gmail.com
Thu Dec 27 21:25:35 EET 2018
2018-12-27 13:00 GMT+01:00, Uwe Freese <uwe.freese at gmx.de>:
> > The "10" should be a #define, [...]
> I have now added as error handling:
> av_log(inlink->src, AV_LOG_ERROR, "More planes in frame than expected.\n");
> return AVERROR(ENOMEM);
> Is this ok, or how should this be implemented instead?
Not sure I understand: How can plane get >= MAX_PLANES?
If this is impossible (as I believe), please use av_assert0().
Please use av_freep() instead of av_free() for non-local
variables (the parameter needs an additional "&").
More information about the ffmpeg-devel