[FFmpeg-devel] [PATCH] simplify crop/pad handling in ffmpeg.c

Luca Abeni lucabe72
Wed Apr 16 13:38:50 CEST 2008


Hi,

Diego Biurrun wrote:
> This is a patch from Herv? Flores to simplify crop/pad handling in
> ffmpeg.c.  For your consideration.
[...]

> Index: ffmpeg.c
> ===================================================================
> --- ffmpeg.c	(revision 12847)
> +++ ffmpeg.c	(working copy)
> @@ -2262,10 +2262,6 @@
>          fprintf(stderr, "Incorrect top crop size\n");
>          av_exit(1);
>      }
> -    if ((frame_topBand % 2) != 0) {
> -        fprintf(stderr, "Top crop size must be a multiple of 2\n");
> -        av_exit(1);
> -    }

Sorry; maybe I am missing something, but I see this check removed from here
and not re-introduced in other places... Do we support cropping and padding
with non-multiple-of-2 quantities?


				Luca




More information about the ffmpeg-devel mailing list