[FFmpeg-devel] [PATCH 2/5] libxvid: add working lumimasking

Timothy Gu timothygu99 at gmail.com
Mon Jul 29 01:12:56 CEST 2013


On Jul 28, 2013 4:07 PM, "Michael Niedermayer" <michaelni at gmx.at> wrote:
>
> On Sun, Jul 28, 2013 at 03:54:52PM -0700, Timothy Gu wrote:
> > On Sunday, July 28, 2013, Michael Niedermayer wrote:
> > >
> > > On Sat, Jul 27, 2013 at 04:32:31PM -0700, Timothy Gu wrote:
[...]
> > > > +static const AVOption options[] = {
> > > > +    { "lumi_masking_mode", "HVS masking mode",
> >  OFFSET(lumi_masking_mode), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, 2, VE,
> > "lumi_masking_mode" },
> > > > +        { "off",           NULL, 0, AV_OPT_TYPE_CONST, { .i64 = 0
},
> > INT_MIN, INT_MAX, VE, "lumi_masking_mode" },
> > > > +        { "lumi",          NULL, 0, AV_OPT_TYPE_CONST, { .i64 = 1
},
> > INT_MIN, INT_MAX, VE, "lumi_masking_mode" },
> > > > +        { "variance",      NULL, 0, AV_OPT_TYPE_CONST, { .i64 = 2
},
> > INT_MIN, INT_MAX, VE, "lumi_masking_mode" },
> > > > +    { NULL },
> > > > +};
> > >
> > > this doesnt allow both to be specified
> >
> > Xvid does not allow both to be specified
>
> i thought one could add 2 plugins one for lumi and one for variance
> masking

No, it is just one big plugin with a switch to use either method.

Timothy


More information about the ffmpeg-devel mailing list