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

[Lukasz Marek]

On 1 November 2014 22:20, Reynaldo H. Verdejo Pinochet <
reynaldo at osg.samsung.com> wrote:

> 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.
>

Can some one confirm it is safe to not demand width and height to be
multiply of 16? Is there any other constraint? like %2?