[FFmpeg-devel] [PATCH] avfilter/vf_delogo: remove duplicated code
Zhao Zhili
quinkblack at foxmail.com
Sat Sep 26 19:38:57 EEST 2020
Ping.
> On Sep 16, 2020, at 1:09 PM, quinkblack at foxmail.com wrote:
>
> From: Zhao Zhili <quinkblack at foxmail.com>
>
> 1. Remove the modification of x, y, w and h parameters since they
> are reset by filter_frame.
> 2. config_input leads to error out when logo area is outside of the
> frame, while filter_frame fix the parameters automatically. Make
> config_input don't return error to keep the logic consistent.
> ---
> libavfilter/vf_delogo.c | 39 +++++++++++++++------------------------
> 1 file changed, 15 insertions(+), 24 deletions(-)
>
> diff --git a/libavfilter/vf_delogo.c b/libavfilter/vf_delogo.c
> index 6069c30163..39f06512fa 100644
> --- a/libavfilter/vf_delogo.c
> +++ b/libavfilter/vf_delogo.c
> @@ -271,11 +271,6 @@ static av_cold int init(AVFilterContext *ctx)
> av_log(ctx, AV_LOG_VERBOSE, "x:%d y:%d, w:%d h:%d band:%d show:%d\n",
> s->x, s->y, s->w, s->h, s->band, s->show);
>
> - s->w += s->band*2;
> - s->h += s->band*2;
> - s->x -= s->band;
> - s->y -= s->band;
> -
> return 0;
> }
>
> @@ -284,10 +279,21 @@ static int config_input(AVFilterLink *inlink)
> DelogoContext *s = inlink->dst->priv;
>
> /* Check whether the logo area fits in the frame */
> - if (s->x + (s->band - 1) < 0 || s->x + s->w - (s->band*2 - 2) > inlink->w ||
> - s->y + (s->band - 1) < 0 || s->y + s->h - (s->band*2 - 2) > inlink->h) {
> - av_log(s, AV_LOG_ERROR, "Logo area is outside of the frame.\n");
> - return AVERROR(EINVAL);
> + if (s->x + (s->band - 1) <= 0 || s->x + s->w - (s->band*2 - 2) > inlink->w ||
> + s->y + (s->band - 1) <= 0 || s->y + s->h - (s->band*2 - 2) > inlink->h) {
> + av_log(s, AV_LOG_WARNING, "Logo area is outside of the frame,"
> + " logo(x:%d y:%d w:%d h:%d), frame(%dx%d),"
> + " auto set the area inside of the frame."
> + " Note: x and y must be 1 at least.\n",
> + s->x, s->y, s->w, s->h, inlink->w, inlink->h);
> + if (s->x + (s->band - 1) <= 0)
> + s->x = 1 + s->band;
> + if (s->y + (s->band - 1) <= 0)
> + s->y = 1 + s->band;
> + if (s->x + s->w - (s->band*2 - 2) > inlink->w)
> + s->w = inlink->w - s->x - (s->band*2 - 2);
> + if (s->y + s->h - (s->band*2 - 2) > inlink->h)
> + s->h = inlink->h - s->y - (s->band*2 - 2);
> }
>
> return 0;
> @@ -313,21 +319,6 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *in)
> s->w = av_expr_eval(s->w_pexpr, s->var_values, s);
> s->h = av_expr_eval(s->h_pexpr, s->var_values, s);
>
> - if (s->x + (s->band - 1) <= 0 || s->x + s->w - (s->band*2 - 2) > inlink->w ||
> - s->y + (s->band - 1) <= 0 || s->y + s->h - (s->band*2 - 2) > inlink->h) {
> - av_log(s, AV_LOG_WARNING, "Logo area is outside of the frame,"
> - " auto set the area inside of the frame\n");
> - }
> -
> - if (s->x + (s->band - 1) <= 0)
> - s->x = 1 + s->band;
> - if (s->y + (s->band - 1) <= 0)
> - s->y = 1 + s->band;
> - if (s->x + s->w - (s->band*2 - 2) > inlink->w)
> - s->w = inlink->w - s->x - (s->band*2 - 2);
> - if (s->y + s->h - (s->band*2 - 2) > inlink->h)
> - s->h = inlink->h - s->y - (s->band*2 - 2);
> -
> ret = config_input(inlink);
> if (ret < 0) {
> av_frame_free(&in);
> --
> 2.17.1
>
>
More information about the ffmpeg-devel
mailing list