[FFmpeg-devel] [PATCH 1/4] ffserver_config: drop requirement video size being multiple of 16

Reynaldo H. Verdejo Pinochet reynaldo at osg.samsung.com
Sat Nov 1 22:20:23 CET 2014


Hi

On 10/31/2014 11:00 PM, Lukasz Marek wrote:
> [..]
> diff --git a/ffserver_config.c b/ffserver_config.c
> index d589ff2..f11db03 100644
> --- a/ffserver_config.c
> +++ b/ffserver_config.c
> @@ -819,8 +819,6 @@ static int ffserver_parse_config_stream(FFServerConfig *config, const char *cmd,
>          ret = av_parse_video_size(&w, &h, arg);
>          if (ret < 0)
>              ERROR("Invalid video size '%s'\n", arg);
> -        else if ((w % 16) || (h % 16))
> -            ERROR("Image size must be a multiple of 16\n");
> [..]

OK if always safe. Otherwise demote to WARNING()

Feel free to push otherwise.

Bests,

-- 
Reynaldo H. Verdejo Pinochet
Open Source Group
Samsung Research America / Silicon Valley


More information about the ffmpeg-devel mailing list