[FFmpeg-devel] [PATCH] lavfi/buffersrc: set min and max values for pix_fmt option

Stefano Sabatini stefasab at gmail.com
Mon Dec 16 18:26:50 CET 2013


On date Monday 2013-12-16 16:41:20 +0100, Michael Niedermayer encoded:
> On Mon, Dec 16, 2013 at 04:04:05PM +0100, Stefano Sabatini wrote:
> > Currently the min and max values are not honored, but this will change in
> > a following patch.
> > ---
> >  libavfilter/buffersrc.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/libavfilter/buffersrc.c b/libavfilter/buffersrc.c
> > index 6b6210d..996251b 100644
> > --- a/libavfilter/buffersrc.c
> > +++ b/libavfilter/buffersrc.c
> > @@ -326,7 +326,7 @@ static const AVOption buffer_options[] = {
> >      { "width",         NULL,                     OFFSET(w),                AV_OPT_TYPE_INT,      { .i64 = 0 }, 0, INT_MAX, V },
> >      { "video_size",    NULL,                     OFFSET(w),                AV_OPT_TYPE_IMAGE_SIZE,                .flags = V },
> >      { "height",        NULL,                     OFFSET(h),                AV_OPT_TYPE_INT,      { .i64 = 0 }, 0, INT_MAX, V },
> > -    { "pix_fmt",       NULL,                     OFFSET(pix_fmt),          AV_OPT_TYPE_PIXEL_FMT,                 .flags = V },
> > +    { "pix_fmt",       NULL,                     OFFSET(pix_fmt),          AV_OPT_TYPE_PIXEL_FMT, { .i64 = 0 }, .min = 0, .max = INT_MAX, .flags = V },
> >  #if FF_API_OLD_FILTER_OPTS
> >      /* those 4 are for compatibility with the old option passing system where each filter
> >       * did its own parsing */
> 
> LGTM

Default and min set to none, applied.
-- 
FFmpeg = Faithful & Foolish Mournful Pitiful Elected Gadget


More information about the ffmpeg-devel mailing list