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

[Benoit Fouet]

Hi,

Diego Biurrun wrote:
> Index: ffmpeg.c
> ===================================================================
> --- ffmpeg.c	(revision 12847)
> +++ ffmpeg.c	(working copy)
> @@ -2965,6 +2929,14 @@
>  
>          video_enc->width = frame_width + frame_padright + frame_padleft;
>          video_enc->height = frame_height + frame_padtop + frame_padbottom;
> +
> +        if ((video_enc->width % 2) != 0 || (video_enc->height % 2) != 0) {
>   

I guess while you're at it, you could remove the "!= 0"

> +            fprintf(stderr, "Error for %s:  Resulting size [%dx%d]"
> +                            "is not a multiple of 2.\n",
> +                    oc->filename, video_enc->width, video_enc->height);
> +            exit(1);
>   

and it may be av_exit(1) here, not exit(1)

-- 
Benoit Fouet
Purple Labs S.A.
www.purplelabs.com