On Thu, Oct 25, 2018 at 12:35 AM Paul B Mahol <one...@gmail.com> wrote:

>
> +        if (s->is_horizontal || s->is_vertical) {
> +            if ((ret = av_image_fill_linesizes(linesize, inlink->format,
> inlink->w)) < 0) {
> +                av_frame_free(&out);
> +                return ret;
> +            }
>

 Nice filter. One unfortunate side-effect is that unused parts of the
canvas are zero-ed which in YUV formats is a shade of dark green. Users
will generally want black. Would be nice to add a color option which fills
the canvas before av_image_copy_plane.

Gyan
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to