[vlc-devel] [RFC] autosaving config options.

Rémi Denis-Courmont rdenis at simphalempin.com
Mon Aug 25 17:35:33 CEST 2008


On Mon, 25 Aug 2008 17:01:55 +0200, Derk-Jan Hartman <hartman at videolan.org>
wrote:
> 1: we remove change_autosave() from the core.
> 2: we introduce something like config_SetAutoSave( optionname, true/
> false) This option will set the flag that change_autosave() used to set.

This fails to address the fundamental flaw. You can still end up with a
configuration change meant to be saved, and one not - at least onto the
same configuration setting.

IMHO, we should be able to set _and_get_[1] either:
- the saved and current configuration (to the same new value),
- or only the current configuration.

[1] so that the UI can display the saved configuration without command line
interference.

And then, there's the persistent lack of clear policy for using config_* vs
var_*.

-- 
Rémi Denis-Courmont




More information about the vlc-devel mailing list