[vlc-devel] [RFC] Option whitelisting policy

[Antoine Cellerier]

On Mon, Sep 22, 2008, Rémi Duraffort wrote:
> For the first patch:
> >          add_bool( "dvb-high-voltage", 0, NULL, HIGH_VOLTAGE_TEXT,
> >              HIGH_VOLTAGE_LONGTEXT, true );
> > +            change_safe();
> I don't like the fact the setting the voltage on the dvb card is safe.

Agreed.

> >      add_integer( "http-caching", 4 * DEFAULT_PTS_DELAY / 1000, NULL,
> >                   CACHING_TEXT, CACHING_LONGTEXT, true );
> > +        change_safe();
> I'm not sure that a good idea to let this option available as safe !!

Why? (either all the caching options are safe, or all are unsafe ... but
there has to be some sort of consistency)

-- 
Antoine Cellerier
dionoea