[vlc-devel] [PATCH] Do not overwrite configuration with command line options

Ludovic Fauvet etix at l0cal.com
Fri Aug 21 04:00:01 CEST 2009


Hi !

These patches tries to fix one of the most problematic misconception
of the VLC's configuration: the non-differentiation of options set by
the command line from those set by the configuration file itself.
The most obvious issue resides in the Qt interface where command line
options are saved on-disk (overwriting user options) everytime the
preferences are saved.
It's been a long time since this problem affect the code base. Besides
this, it also prevents some functionalities to work like the
qt-autosave-volume which can't be activated without risking to save
command line options each time VLC is closed.

The fix is composed of exactly 4 patches that affects the core (to be
easier to review) and will be followed by some more in the upcoming
days.

Comments are appreciated.

Best regards,

-- 
Ludovic Fauvet
| Jabber : etix at im.l0cal.com
| GPG    : 1024D/BB558EED
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Config-New-module_value_t-to-save-command-line-argum.patch
Type: text/x-patch
Size: 31748 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20090821/3cbf887b/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Config-Create-advanced-functions-for-reading-configu.patch
Type: text/x-patch
Size: 6475 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20090821/3cbf887b/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-Config-Create-advanced-functions-to-save-configurati.patch
Type: text/x-patch
Size: 11735 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20090821/3cbf887b/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-Config-libvlc_audio_output_device_count-adjustments.patch
Type: text/x-patch
Size: 2374 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20090821/3cbf887b/attachment-0003.bin>


More information about the vlc-devel mailing list