]> git.sesse.net Git - vlc/blobdiff - src/misc/configuration.c
A bit of cleanup here and there
[vlc] / src / misc / configuration.c
index 3c4787454aa384d116d1f5e24240edeb211dec7e..a36fc1b76c66d9e6d556905459aea1cf0fe8a0c8 100644 (file)
@@ -1,7 +1,7 @@
 /*****************************************************************************
  * configuration.c management of the modules configuration
  *****************************************************************************
- * Copyright (C) 2001-2004 VideoLAN
+ * Copyright (C) 2001-2004 the VideoLAN team
  * $Id$
  *
  * Authors: Gildas Bazin <gbazin@videolan.org>
  *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111, USA.
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
  *****************************************************************************/
 
 #include <vlc/vlc.h>
 #include "vlc_keys.h"
+#include "vlc_charset.h"
 
 #include <stdio.h>                                              /* sprintf() */
 #include <stdlib.h>                                      /* free(), strtol() */
 #include <string.h>                                              /* strdup() */
 #include <errno.h>                                                  /* errno */
 
+#ifdef HAVE_LIMITS_H
+#   include <limits.h>
+#endif
+
 #ifdef HAVE_UNISTD_H
 #    include <unistd.h>                                          /* getuid() */
 #endif
 #include <tchar.h>
 #endif
 
+#include "configuration.h"
 
-static int ConfigStringToKey( char * );
+static int ConfigStringToKey( const char * );
 static char *ConfigKeyToString( int );
 
+static inline void freenull (const void *p)
+{
+    if (p != NULL)
+        free ((void *)p);
+}
+
+static inline char *strdupnull (const char *src)
+{
+    if (src == NULL)
+        return NULL;
+    return strdup (src);
+}
+
+static inline char *_strdupnull (const char *src)
+{
+    if (src == NULL)
+        return NULL;
+    return strdup (_(src));
+}
+
+
 /*****************************************************************************
  * config_GetType: get the type of a variable (bool, int, float, string)
  *****************************************************************************
@@ -96,6 +123,9 @@ int __config_GetType( vlc_object_t *p_this, const char *psz_name )
         break;
 
     case CONFIG_ITEM_MODULE:
+    case CONFIG_ITEM_MODULE_CAT:
+    case CONFIG_ITEM_MODULE_LIST:
+    case CONFIG_ITEM_MODULE_LIST_CAT:
         i_type = VLC_VAR_MODULE;
         break;
 
@@ -138,15 +168,14 @@ int __config_GetInt( vlc_object_t *p_this, const char *psz_name )
         msg_Err( p_this, "option %s does not exist", psz_name );
         return -1;
     }
-    if( (p_config->i_type!=CONFIG_ITEM_INTEGER) &&
-        (p_config->i_type!=CONFIG_ITEM_KEY) &&
-        (p_config->i_type!=CONFIG_ITEM_BOOL) )
+
+    if (!IsConfigIntegerType (p_config->i_type))
     {
         msg_Err( p_this, "option %s does not refer to an int", psz_name );
         return -1;
     }
 
-    return p_config->i_value;
+    return p_config->value.i;
 }
 
 /*****************************************************************************
@@ -167,13 +196,14 @@ float __config_GetFloat( vlc_object_t *p_this, const char *psz_name )
         msg_Err( p_this, "option %s does not exist", psz_name );
         return -1;
     }
-    if( p_config->i_type != CONFIG_ITEM_FLOAT )
+
+    if (!IsConfigFloatType (p_config->i_type))
     {
         msg_Err( p_this, "option %s does not refer to a float", psz_name );
         return -1;
     }
 
-    return p_config->f_value;
+    return p_config->value.f;
 }
 
 /*****************************************************************************
@@ -190,7 +220,6 @@ float __config_GetFloat( vlc_object_t *p_this, const char *psz_name )
 char * __config_GetPsz( vlc_object_t *p_this, const char *psz_name )
 {
     module_config_t *p_config;
-    char *psz_value = NULL;
 
     p_config = config_FindConfig( p_this, psz_name );
 
@@ -200,13 +229,8 @@ char * __config_GetPsz( vlc_object_t *p_this, const char *psz_name )
         msg_Err( p_this, "option %s does not exist", psz_name );
         return NULL;
     }
-    if( (p_config->i_type!=CONFIG_ITEM_STRING) &&
-        (p_config->i_type!=CONFIG_ITEM_FILE) &&
-        (p_config->i_type!=CONFIG_ITEM_DIRECTORY) &&
-        (p_config->i_type!=CONFIG_ITEM_MODULE_LIST) &&
-        (p_config->i_type!=CONFIG_ITEM_MODULE_LIST_CAT) &&
-        (p_config->i_type!=CONFIG_ITEM_MODULE_CAT) &&
-        (p_config->i_type!=CONFIG_ITEM_MODULE) )
+
+    if (!IsConfigStringType (p_config->i_type))
     {
         msg_Err( p_this, "option %s does not refer to a string", psz_name );
         return NULL;
@@ -214,7 +238,7 @@ char * __config_GetPsz( vlc_object_t *p_this, const char *psz_name )
 
     /* return a copy of the string */
     vlc_mutex_lock( p_config->p_lock );
-    if( p_config->psz_value ) psz_value = strdup( p_config->psz_value );
+    char *psz_value = strdupnull (p_config->value.psz);
     vlc_mutex_unlock( p_config->p_lock );
 
     return psz_value;
@@ -235,18 +259,15 @@ void __config_PutPsz( vlc_object_t *p_this,
 
     p_config = config_FindConfig( p_this, psz_name );
 
+
     /* sanity checks */
     if( !p_config )
     {
         msg_Warn( p_this, "option %s does not exist", psz_name );
         return;
     }
-    if( (p_config->i_type!=CONFIG_ITEM_STRING) &&
-        (p_config->i_type!=CONFIG_ITEM_FILE) &&
-        (p_config->i_type!=CONFIG_ITEM_DIRECTORY) &&
-        (p_config->i_type!=CONFIG_ITEM_MODULE_LIST) &&
-        (p_config->i_type!=CONFIG_ITEM_MODULE_LIST_CAT) &&
-        (p_config->i_type!=CONFIG_ITEM_MODULE) )
+
+    if (!IsConfigStringType (p_config->i_type))
     {
         msg_Err( p_this, "option %s does not refer to a string", psz_name );
         return;
@@ -255,12 +276,16 @@ void __config_PutPsz( vlc_object_t *p_this,
     vlc_mutex_lock( p_config->p_lock );
 
     /* backup old value */
-    oldval.psz_string = p_config->psz_value;
+    oldval.psz_string = (char *)p_config->value.psz;
 
-    if( psz_value && *psz_value ) p_config->psz_value = strdup( psz_value );
-    else p_config->psz_value = NULL;
+    if ((psz_value != NULL) && *psz_value)
+        p_config->value.psz = strdup (psz_value);
+    else
+        p_config->value.psz = NULL;
+
+    p_config->b_dirty = VLC_TRUE;
 
-    val.psz_string = p_config->psz_value;
+    val.psz_string = (char *)p_config->value.psz;
 
     vlc_mutex_unlock( p_config->p_lock );
 
@@ -294,36 +319,37 @@ void __config_PutInt( vlc_object_t *p_this, const char *psz_name, int i_value )
         msg_Warn( p_this, "option %s does not exist", psz_name );
         return;
     }
-    if( (p_config->i_type!=CONFIG_ITEM_INTEGER) &&
-        (p_config->i_type!=CONFIG_ITEM_KEY) &&
-        (p_config->i_type!=CONFIG_ITEM_BOOL) )
+
+    if (!IsConfigIntegerType (p_config->i_type))
     {
         msg_Err( p_this, "option %s does not refer to an int", psz_name );
         return;
     }
 
     /* backup old value */
-    oldval.i_int = p_config->i_value;
+    oldval.i_int = p_config->value.i;
 
     /* if i_min == i_max == 0, then do not use them */
-    if ((p_config->i_min == 0) && (p_config->i_max == 0))
+    if ((p_config->min.i == 0) && (p_config->max.i == 0))
     {
-        p_config->i_value = i_value;
+        p_config->value.i = i_value;
     }
-    else if (i_value < p_config->i_min)
+    else if (i_value < p_config->min.i)
     {
-        p_config->i_value = p_config->i_min;
+        p_config->value.i = p_config->min.i;
     }
-    else if (i_value > p_config->i_max)
+    else if (i_value > p_config->max.i)
     {
-        p_config->i_value = p_config->i_max;
+        p_config->value.i = p_config->max.i;
     }
     else
     {
-        p_config->i_value = i_value;
+        p_config->value.i = i_value;
     }
 
-    val.i_int = p_config->i_value;
+    p_config->b_dirty = VLC_TRUE;
+
+    val.i_int = p_config->value.i;
 
     if( p_config->pf_callback )
     {
@@ -352,34 +378,37 @@ void __config_PutFloat( vlc_object_t *p_this,
         msg_Warn( p_this, "option %s does not exist", psz_name );
         return;
     }
-    if( p_config->i_type != CONFIG_ITEM_FLOAT )
+
+    if (!IsConfigFloatType (p_config->i_type))
     {
         msg_Err( p_this, "option %s does not refer to a float", psz_name );
         return;
     }
 
     /* backup old value */
-    oldval.f_float = p_config->f_value;
+    oldval.f_float = p_config->value.f;
 
     /* if f_min == f_max == 0, then do not use them */
-    if ((p_config->f_min == 0) && (p_config->f_max == 0))
+    if ((p_config->min.f == 0) && (p_config->max.f == 0))
     {
-        p_config->f_value = f_value;
+        p_config->value.f = f_value;
     }
-    else if (f_value < p_config->f_min)
+    else if (f_value < p_config->min.f)
     {
-        p_config->f_value = p_config->f_min;
+        p_config->value.f = p_config->min.f;
     }
-    else if (f_value > p_config->f_max)
+    else if (f_value > p_config->max.f)
     {
-        p_config->f_value = p_config->f_max;
+        p_config->value.f = p_config->max.f;
     }
     else
     {
-        p_config->f_value = f_value;
+        p_config->value.f = f_value;
     }
 
-    val.f_float = p_config->f_value;
+    p_config->b_dirty = VLC_TRUE;
+
+    val.f_float = p_config->value.f;
 
     if( p_config->pf_callback )
     {
@@ -397,8 +426,6 @@ void __config_PutFloat( vlc_object_t *p_this,
 module_config_t *config_FindConfig( vlc_object_t *p_this, const char *psz_name )
 {
     vlc_list_t *p_list;
-    module_t *p_parser;
-    module_config_t *p_item;
     int i_index;
 
     if( !psz_name ) return NULL;
@@ -407,13 +434,14 @@ module_config_t *config_FindConfig( vlc_object_t *p_this, const char *psz_name )
 
     for( i_index = 0; i_index < p_list->i_count; i_index++ )
     {
-        p_parser = (module_t *)p_list->p_values[i_index].p_object ;
+        module_config_t *p_item, *p_end;
+        module_t *p_parser = (module_t *)p_list->p_values[i_index].p_object;
 
         if( !p_parser->i_config_items )
             continue;
 
-        for( p_item = p_parser->p_config;
-             p_item->i_type != CONFIG_HINT_END;
+        for( p_item = p_parser->p_config, p_end = p_item + p_parser->confsize;
+             p_item < p_end;
              p_item++ )
         {
             if( p_item->i_type & CONFIG_HINT )
@@ -467,19 +495,18 @@ module_t *config_FindModule( vlc_object_t *p_this, const char *psz_name )
  * this module might be unloaded from memory at any time (remember HideModule).
  * This is why we need to create an exact copy of the config data.
  *****************************************************************************/
-void config_Duplicate( module_t *p_module, module_config_t *p_orig )
+int config_Duplicate( module_t *p_module, const module_config_t *p_orig,
+                      size_t n )
 {
-    int i, j, i_lines = 1;
-    module_config_t *p_item;
+    int j;
+    const module_config_t *p_item, *p_end = p_orig + n;
 
     /* Calculate the structure length */
     p_module->i_config_items = 0;
     p_module->i_bool_items = 0;
 
-    for( p_item = p_orig; p_item->i_type != CONFIG_HINT_END; p_item++ )
+    for( p_item = p_orig; p_item < p_end; p_item++ )
     {
-        i_lines++;
-
         if( p_item->i_type & CONFIG_ITEM )
         {
             p_module->i_config_items++;
@@ -492,34 +519,43 @@ void config_Duplicate( module_t *p_module, module_config_t *p_orig )
     }
 
     /* Allocate memory */
-    p_module->p_config = (module_config_t *)malloc( sizeof(module_config_t)
-                                                     * i_lines );
+    p_module->p_config = (module_config_t *)calloc( n, sizeof(*p_orig) );
     if( p_module->p_config == NULL )
     {
         msg_Err( p_module, "config error: can't duplicate p_config" );
-        return;
+        return VLC_ENOMEM;
     }
+    p_module->confsize = n;
 
     /* Do the duplication job */
-    for( i = 0; i < i_lines ; i++ )
+    for( size_t i = 0; i < n ; i++ )
     {
         p_module->p_config[i] = p_orig[i];
 
-        p_module->p_config[i].i_value_orig = p_orig[i].i_value;
-        p_module->p_config[i].f_value_orig = p_orig[i].f_value;
-
-        p_module->p_config[i].psz_type = p_orig[i].psz_type ?
-                                   strdup( p_orig[i].psz_type ) : NULL;
-        p_module->p_config[i].psz_name = p_orig[i].psz_name ?
-                                   strdup( p_orig[i].psz_name ) : NULL;
-        p_module->p_config[i].psz_text = p_orig[i].psz_text ?
-                                   strdup( _(p_orig[i].psz_text) ) : NULL;
-        p_module->p_config[i].psz_longtext = p_orig[i].psz_longtext ?
-                                   strdup( _(p_orig[i].psz_longtext) ) : NULL;
-        p_module->p_config[i].psz_value = p_orig[i].psz_value ?
-                                   strdup( p_orig[i].psz_value ) : NULL;
-        p_module->p_config[i].psz_value_orig = p_orig[i].psz_value ?
-                                   strdup( p_orig[i].psz_value ) : NULL;
+        if (IsConfigIntegerType (p_module->p_config[i].i_type))
+        {
+            p_module->p_config[i].orig.i = p_orig[i].value.i;
+            p_module->p_config[i].saved.i = p_orig[i].value.i;
+        }
+        else
+        if (IsConfigFloatType (p_module->p_config[i].i_type))
+        {
+            p_module->p_config[i].orig.f = p_orig[i].value.f;
+            p_module->p_config[i].saved.f = p_orig[i].value.f;
+        }
+        else
+        if (IsConfigStringType (p_module->p_config[i].i_type))
+        {
+            p_module->p_config[i].value.psz = strdupnull (p_orig[i].value.psz);
+            p_module->p_config[i].orig.psz = strdupnull (p_orig[i].value.psz);
+            p_module->p_config[i].saved.psz = NULL;
+        }
+
+        p_module->p_config[i].psz_type = strdupnull (p_orig[i].psz_type);
+        p_module->p_config[i].psz_name = strdupnull (p_orig[i].psz_name);
+        p_module->p_config[i].psz_current = strdupnull (p_orig[i].psz_current);
+        p_module->p_config[i].psz_text = _strdupnull (p_orig[i].psz_text);
+        p_module->p_config[i].psz_longtext = _strdupnull (p_orig[i].psz_longtext);
 
         p_module->p_config[i].p_lock = &p_module->object_lock;
 
@@ -533,20 +569,20 @@ void config_Duplicate( module_t *p_module, module_config_t *p_orig )
                 if( p_module->p_config[i].ppsz_list )
                 {
                     for( j = 0; j < p_orig[i].i_list; j++ )
-                        p_module->p_config[i].ppsz_list[j] = p_orig[i].ppsz_list[j] ?
-                            strdup( p_orig[i].ppsz_list[j] ) : NULL ;
+                        p_module->p_config[i].ppsz_list[j] =
+                                strdupnull (p_orig[i].ppsz_list[j]);
                     p_module->p_config[i].ppsz_list[j] = NULL;
                 }
             }
             if( p_orig[i].ppsz_list_text )
             {
                 p_module->p_config[i].ppsz_list_text =
-                    malloc( (p_orig[i].i_list + 1) * sizeof(char *) );
+                    calloc( (p_orig[i].i_list + 1), sizeof(char *) );
                 if( p_module->p_config[i].ppsz_list_text )
                 {
                     for( j = 0; j < p_orig[i].i_list; j++ )
-                        p_module->p_config[i].ppsz_list_text[j] = _(p_orig[i].ppsz_list_text[j]) ?
-                            strdup( _(p_orig[i].ppsz_list_text[j] ) ) : NULL ;
+                        p_module->p_config[i].ppsz_list_text[j] =
+                                strdupnull (_(p_orig[i].ppsz_list_text[j]));
                     p_module->p_config[i].ppsz_list_text[j] = NULL;
                 }
             }
@@ -578,15 +614,16 @@ void config_Duplicate( module_t *p_module, module_config_t *p_orig )
                 p_module->p_config[i].ppf_action[j] =
                     p_orig[i].ppf_action[j];
                 p_module->p_config[i].ppsz_action_text[j] =
-                    p_orig[i].ppsz_action_text[j] ?
-                    strdup( p_orig[i].ppsz_action_text[j] ) : NULL;
+                    strdupnull (p_orig[i].ppsz_action_text[j]);
             }
         }
 
         p_module->p_config[i].pf_callback = p_orig[i].pf_callback;
     }
+    return VLC_SUCCESS;
 }
 
+
 /*****************************************************************************
  * config_Free: frees a duplicated module's configuration data.
  *****************************************************************************
@@ -594,42 +631,33 @@ void config_Duplicate( module_t *p_module, module_config_t *p_orig )
  *****************************************************************************/
 void config_Free( module_t *p_module )
 {
-    module_config_t *p_item = p_module->p_config;
     int i;
 
-    if( p_item == NULL )
-    {
-        return;
-    }
-
-    for( ; p_item->i_type != CONFIG_HINT_END ; p_item++ )
+    for (size_t j = 0; j < p_module->confsize; j++)
     {
-        if( p_item->psz_type )
-            free( p_item->psz_type );
-
-        if( p_item->psz_name )
-            free( p_item->psz_name );
-
-        if( p_item->psz_text )
-            free( p_item->psz_text );
+        module_config_t *p_item = p_module->p_config + j;
 
-        if( p_item->psz_longtext )
-            free( p_item->psz_longtext );
+        free( (char*) p_item->psz_type );
+        free( (char*) p_item->psz_name );
+        free( (char*) p_item->psz_current );
+        free( (char*) p_item->psz_text );
+        free( (char*) p_item->psz_longtext );
 
-        if( p_item->psz_value )
-            free( p_item->psz_value );
-
-        if( p_item->psz_value_orig )
-            free( p_item->psz_value_orig );
+        if (IsConfigStringType (p_item->i_type))
+        {
+            freenull (p_item->value.psz);
+            freenull (p_item->orig.psz);
+            freenull (p_item->saved.psz);
+        }
 
         if( p_item->i_list )
         {
             for( i = 0; i < p_item->i_list; i++ )
             {
                 if( p_item->ppsz_list && p_item->ppsz_list[i] )
-                    free( p_item->ppsz_list[i] );
+                    free( (char*) p_item->ppsz_list[i] );
                 if( p_item->ppsz_list_text && p_item->ppsz_list_text[i] )
-                    free( p_item->ppsz_list_text[i] );
+                    free( (char*) p_item->ppsz_list_text[i] );
             }
             if( p_item->ppsz_list ) free( p_item->ppsz_list );
             if( p_item->ppsz_list_text ) free( p_item->ppsz_list_text );
@@ -640,16 +668,18 @@ void config_Free( module_t *p_module )
         {
             for( i = 0; i < p_item->i_action; i++ )
             {
-                if( p_item->ppsz_action_text[i] )
-                    free( p_item->ppsz_action_text[i] );
+                free( (char*) p_item->ppsz_action_text[i] );
             }
             if( p_item->ppf_action ) free( p_item->ppf_action );
             if( p_item->ppsz_action_text ) free( p_item->ppsz_action_text );
         }
     }
 
-    free( p_module->p_config );
-    p_module->p_config = NULL;
+    if (p_module->p_config != NULL)
+    {
+        free (p_module->p_config);
+        p_module->p_config = NULL;
+    }
 }
 
 /*****************************************************************************
@@ -659,9 +689,10 @@ void config_Free( module_t *p_module )
  * this module might be unloaded from memory at any time (remember HideModule).
  * This is why we need to duplicate callbacks each time we reload the module.
  *****************************************************************************/
-void config_SetCallbacks( module_config_t *p_new, module_config_t *p_orig )
+void config_SetCallbacks( module_config_t *p_new, module_config_t *p_orig,
+                          size_t n )
 {
-    while( p_new->i_type != CONFIG_HINT_END )
+    for (size_t i = 0; i < n; i++)
     {
         p_new->pf_callback = p_orig->pf_callback;
         p_new++;
@@ -674,9 +705,9 @@ void config_SetCallbacks( module_config_t *p_new, module_config_t *p_orig )
  *****************************************************************************
  * We simply undo what we did in config_SetCallbacks.
  *****************************************************************************/
-void config_UnsetCallbacks( module_config_t *p_new )
+void config_UnsetCallbacks( module_config_t *p_new, size_t n )
 {
-    while( p_new->i_type != CONFIG_HINT_END )
+    for (size_t i = 0; i < n; i++)
     {
         p_new->pf_callback = NULL;
         p_new++;
@@ -688,12 +719,12 @@ void config_UnsetCallbacks( module_config_t *p_new )
  *****************************************************************************/
 void __config_ResetAll( vlc_object_t *p_this )
 {
-    int i_index, i;
+    int i_index;
     vlc_list_t *p_list;
     module_t *p_module;
 
     /* Acquire config file lock */
-    vlc_mutex_lock( &p_this->p_vlc->config_lock );
+    vlc_mutex_lock( &p_this->p_libvlc->config_lock );
 
     p_list = vlc_list_find( p_this, VLC_OBJECT_MODULE, FIND_ANYWHERE );
 
@@ -702,22 +733,63 @@ void __config_ResetAll( vlc_object_t *p_this )
         p_module = (module_t *)p_list->p_values[i_index].p_object ;
         if( p_module->b_submodule ) continue;
 
-        for( i = 0; p_module->p_config[i].i_type != CONFIG_HINT_END; i++ )
+        for (size_t i = 0; i < p_module->confsize; i++ )
         {
-            p_module->p_config[i].i_value = p_module->p_config[i].i_value_orig;
-            p_module->p_config[i].f_value = p_module->p_config[i].f_value_orig;
-            if( p_module->p_config[i].psz_value )
-                free( p_module->p_config[i].psz_value );
-            p_module->p_config[i].psz_value =
-                p_module->p_config[i].psz_value_orig ?
-                strdup( p_module->p_config[i].psz_value_orig ) : NULL;
+            if (IsConfigIntegerType (p_module->p_config[i].i_type))
+                p_module->p_config[i].value.i = p_module->p_config[i].orig.i;
+            else
+            if (IsConfigFloatType (p_module->p_config[i].i_type))
+                p_module->p_config[i].value.f = p_module->p_config[i].orig.f;
+            else
+            if (IsConfigStringType (p_module->p_config[i].i_type))
+            {
+                freenull (p_module->p_config[i].value.psz);
+                p_module->p_config[i].value.psz =
+                        strdupnull (p_module->p_config[i].orig.psz);
+            }
         }
     }
 
     vlc_list_release( p_list );
-    vlc_mutex_unlock( &p_this->p_vlc->config_lock );
+    vlc_mutex_unlock( &p_this->p_libvlc->config_lock );
 }
 
+
+static FILE *config_OpenConfigFile (vlc_object_t *obj, const char *mode)
+{
+    static const char subpath[] = DIR_SEP CONFIG_DIR DIR_SEP CONFIG_FILE;
+    const char *filename = obj->p_libvlc->psz_configfile;
+    const char *homedir;
+    size_t buflen = 0;
+
+    if (filename == NULL)
+    {
+        homedir = obj->p_libvlc->psz_homedir;
+        if (homedir == NULL)
+        {
+            msg_Err (obj, "no home directory defined");
+            return NULL;
+        }
+
+        buflen = strlen (homedir) + sizeof (subpath);
+    }
+
+    char buf[buflen];
+    if (filename == NULL)
+    {
+        sprintf (buf, "%s%s", homedir, subpath);
+        filename = buf;
+    }
+
+    msg_Dbg (obj, "opening config file (%s)", filename);
+    FILE *stream = utf8_fopen (filename, mode);
+    if ((stream == NULL) && (errno != ENOENT))
+        msg_Err (obj, "cannot open config file (%s): %s", strerror (errno));
+
+    return stream;
+}
+
+
 /*****************************************************************************
  * config_LoadConfigFile: loads the configuration file.
  *****************************************************************************
@@ -727,234 +799,178 @@ void __config_ResetAll( vlc_object_t *p_this )
 int __config_LoadConfigFile( vlc_object_t *p_this, const char *psz_module_name )
 {
     vlc_list_t *p_list;
-    module_t *p_parser;
-    module_config_t *p_item;
     FILE *file;
-    char line[1024];
-    char *p_index, *psz_option_name, *psz_option_value;
-    char *psz_filename, *psz_homedir, *psz_configfile;
-    int i_index;
 
-    psz_configfile = p_this->p_vlc->psz_configfile;
-    if( !psz_configfile || !psz_configfile )
-    {
-        psz_homedir = p_this->p_vlc->psz_homedir;
-        if( !psz_homedir )
-        {
-            msg_Err( p_this, "psz_homedir is null" );
-            return -1;
-        }
-        psz_filename = (char *)malloc( sizeof("/" CONFIG_DIR "/" CONFIG_FILE) +
-                                       strlen(psz_homedir) );
-        if( psz_filename )
-            sprintf( psz_filename, "%s/" CONFIG_DIR "/" CONFIG_FILE,
-                     psz_homedir );
-    }
-    else
-    {
-        psz_filename = strdup( psz_configfile );
-    }
+    file = config_OpenConfigFile (p_this, "rt");
+    if (file == NULL)
+        return VLC_EGENERIC;
 
-    if( !psz_filename )
-    {
-        msg_Err( p_this, "out of memory" );
-        return -1;
-    }
+    /* Acquire config file lock */
+    vlc_mutex_lock( &p_this->p_libvlc->config_lock );
 
-    msg_Dbg( p_this, "opening config file %s", psz_filename );
+    /* Look for the selected module, if NULL then save everything */
+    p_list = vlc_list_find( p_this, VLC_OBJECT_MODULE, FIND_ANYWHERE );
 
-    /* Acquire config file lock */
-    vlc_mutex_lock( &p_this->p_vlc->config_lock );
+    /* Look for UTF-8 Byte Order Mark */
+    char * (*convert) (const char *) = strdupnull;
+    char bom[3];
 
-    file = fopen( psz_filename, "rt" );
-    if( !file )
+    if ((fread (bom, 1, 3, file) != 3)
+     || memcmp (bom, "\xEF\xBB\xBF", 3))
     {
-        msg_Warn( p_this, "config file %s does not exist yet", psz_filename );
-        free( psz_filename );
-        vlc_mutex_unlock( &p_this->p_vlc->config_lock );
-        return -1;
+        convert = FromLocaleDup;
+        rewind (file); // no BOM, rewind
     }
 
-    /* Look for the selected module, if NULL then save everything */
-    p_list = vlc_list_find( p_this, VLC_OBJECT_MODULE, FIND_ANYWHERE );
+    module_t *module = NULL;
+    char line[1024], section[1022];
+    section[0] = '\0';
 
-    for( i_index = 0; i_index < p_list->i_count; i_index++ )
+    while (fgets (line, 1024, file) != NULL)
     {
-        p_parser = (module_t *)p_list->p_values[i_index].p_object ;
-
-        if( psz_module_name
-             && strcmp( psz_module_name, p_parser->psz_object_name ) )
+        // Ignore comments and empty lines
+        switch (line[0])
         {
-            continue;
+            case '#':
+            case '\n':
+            case '\0':
+                continue;
         }
 
-        /* The config file is organized in sections, one per module. Look for
-         * the interesting section ( a section is of the form [foo] ) */
-        fseek( file, 0L, SEEK_SET );
-        while( fgets( line, 1024, file ) )
+        if (line[0] == '[')
         {
-            if( (line[0] == '[')
-               && (p_index = strchr(line,']'))
-               && (p_index - &line[1]
-                    == (int)strlen(p_parser->psz_object_name))
-               && !memcmp( &line[1], p_parser->psz_object_name,
-                           strlen(p_parser->psz_object_name) ) )
+            char *ptr = strchr (line, ']');
+            if (ptr == NULL)
+                continue; // syntax error;
+            *ptr = '\0';
+
+            // New section ( = a given module)
+            strcpy (section, line + 1);
+            module = NULL;
+
+            if ((psz_module_name == NULL)
+             || (strcmp (psz_module_name, section) == 0))
             {
-#if 0
-                msg_Dbg( p_this, "loading config for module \"%s\"",
-                                 p_parser->psz_object_name );
-#endif
+                for (int i = 0; i < p_list->i_count; i++)
+                {
+                    module_t *m = (module_t *)p_list->p_values[i].p_object;
 
-                break;
+                    if ((strcmp (section, m->psz_object_name) == 0)
+                     && (m->i_config_items > 0)) // ignore config-less modules
+                    {
+                        module = m;
+                        if (psz_module_name != NULL)
+                            msg_Dbg (p_this,
+                                     "loading config for module \"%s\"",
+                                     section);
+                        break;
+                    }
+                }
             }
+
+            continue;
         }
-        /* either we found the section or we're at the EOF */
 
-        /* Now try to load the options in this section */
-        while( fgets( line, 1024, file ) )
-        {
-            if( line[0] == '[' ) break; /* end of section */
+        if (module == NULL)
+            continue; // no need to parse if there is no matching module
 
-            /* ignore comments or empty lines */
-            if( (line[0] == '#') || (line[0] == '\n') || (line[0] == (char)0) )
-                continue;
+        char *ptr = strchr (line, '\n');
+        if (ptr != NULL)
+            *ptr = '\0';
 
-            /* get rid of line feed */
-            if( line[strlen(line)-1] == '\n' )
-                line[strlen(line)-1] = (char)0;
+        /* look for option name */
+        const char *psz_option_name = line;
 
-            /* look for option name */
-            psz_option_name = line;
-            psz_option_value = NULL;
-            p_index = strchr( line, '=' );
-            if( !p_index ) break; /* this ain't an option!!! */
+        ptr = strchr (line, '=');
+        if (ptr == NULL)
+            continue; // syntax error
 
-            *p_index = (char)0;
-            psz_option_value = p_index + 1;
+        *ptr = '\0';
+        const char *psz_option_value = ptr + 1;
 
-            if( !p_parser->i_config_items )
-            {
+        /* try to match this option with one of the module's options */
+        for (size_t i = 0; i < module->confsize; i++)
+        {
+            module_config_t *p_item = module->p_config + i;
+
+            if ((p_item->i_type & CONFIG_HINT)
+             || strcmp (p_item->psz_name, psz_option_name))
                 continue;
-            }
 
-            /* try to match this option with one of the module's options */
-            for( p_item = p_parser->p_config;
-                 p_item->i_type != CONFIG_HINT_END;
-                 p_item++ )
+            /* We found it */
+            switch( p_item->i_type )
             {
-                if( p_item->i_type & CONFIG_HINT )
-                    /* ignore hints */
-                    continue;
-
-                if( !strcmp( p_item->psz_name, psz_option_name ) )
-                {
-                    /* We found it */
-                    switch( p_item->i_type )
-                    {
-                    case CONFIG_ITEM_BOOL:
-                    case CONFIG_ITEM_INTEGER:
-                        if( !*psz_option_value )
-                            break;                    /* ignore empty option */
-                        p_item->i_value = strtol( psz_option_value, 0, 0 );
-#if 0
-                        msg_Dbg( p_this, "option \"%s\", value %i",
-                                 p_item->psz_name, p_item->i_value );
-#endif
-                        break;
-
-                    case CONFIG_ITEM_FLOAT:
-                        if( !*psz_option_value )
-                            break;                    /* ignore empty option */
-                        p_item->f_value = (float)atof( psz_option_value);
-#if O
-                        msg_Dbg( p_this, "option \"%s\", value %f",
-                                 p_item->psz_name, (double)p_item->f_value );
-#endif
-                        break;
-                    case CONFIG_ITEM_KEY:
-                        if( !*psz_option_value )
-                            break;                    /* ignore empty option */
-                        p_item->i_value = ConfigStringToKey( psz_option_value );
-                        break;
-
-                    default:
-                        vlc_mutex_lock( p_item->p_lock );
-
-                        /* free old string */
-                        if( p_item->psz_value )
-                            free( p_item->psz_value );
-
-                        p_item->psz_value = *psz_option_value ?
-                            strdup( psz_option_value ) : NULL;
-
-                        vlc_mutex_unlock( p_item->p_lock );
-
-#if 0
-                        msg_Dbg( p_this, "option \"%s\", value \"%s\"",
-                                 p_item->psz_name,
-                                 p_item->psz_value ? p_item->psz_value : "" );
-#endif
-                        break;
-                    }
-                }
+                case CONFIG_ITEM_BOOL:
+                case CONFIG_ITEM_INTEGER:
+                    if( !*psz_option_value )
+                        break;                    /* ignore empty option */
+                    p_item->value.i = strtol( psz_option_value, 0, 0 );
+                    p_item->saved.i = p_item->value.i;
+
+                    /*msg_Dbg (p_this, "option \"%s\", value %i",
+                             psz_option_name, p_item->value.i);*/
+                    break;
+
+                case CONFIG_ITEM_FLOAT:
+                    if( !*psz_option_value )
+                        break;                    /* ignore empty option */
+                    p_item->value.f = (float)i18n_atof( psz_option_value);
+                    p_item->saved.f = p_item->value.f;
+
+                    /*msg_Dbg (p_this, "option \"%s\", value %f",
+                             psz_option_name, (double)p_item->value.f);*/
+                    break;
+
+                case CONFIG_ITEM_KEY:
+                    if( !*psz_option_value )
+                        break;                    /* ignore empty option */
+                    p_item->value.i = ConfigStringToKey(psz_option_value);
+                    p_item->saved.i = p_item->value.i;
+                    break;
+
+                default:
+                    vlc_mutex_lock( p_item->p_lock );
+
+                    /* free old string */
+                    free( (char*) p_item->value.psz );
+                    free( (char*) p_item->saved.psz );
+
+                    p_item->value.psz = convert (psz_option_value);
+                    p_item->saved.psz = strdupnull (p_item->value.psz);
+
+                    vlc_mutex_unlock( p_item->p_lock );
+
+                    /*msg_Dbg (p_this, "option \"%s\", value \"%s\"",
+                             psz_option_name, psz_option_value ?: "");*/
+                    break;
             }
-        }
 
+            break;
+        }
     }
 
     vlc_list_release( p_list );
 
     fclose( file );
-    free( psz_filename );
-
-    vlc_mutex_unlock( &p_this->p_vlc->config_lock );
 
+    vlc_mutex_unlock( &p_this->p_libvlc->config_lock );
     return 0;
 }
 
 /*****************************************************************************
  * config_CreateDir: Create configuration directory if it doesn't exist.
  *****************************************************************************/
-int config_CreateDir( vlc_object_t *p_this, char *psz_dirname )
+int config_CreateDir( vlc_object_t *p_this, const char *psz_dirname )
 {
     if( !psz_dirname && !*psz_dirname ) return -1;
 
-#if defined( UNDER_CE )
-    {
-        wchar_t psz_new[ MAX_PATH ];
-        char psz_mod[MAX_PATH];
-        int i = 0;
-
-        /* Convert '/' into '\' */
-        while( *psz_dirname )
-        {
-            if( *psz_dirname == '/' ) psz_mod[i] = '\\';
-            else psz_mod[i] = *psz_dirname;
-            psz_dirname++;
-            i++;
-        }
-        psz_mod[i] = 0;
-
-        MultiByteToWideChar( CP_ACP, 0, psz_mod, -1, psz_new, MAX_PATH );
-        if( CreateDirectory( psz_new, NULL ) )
-        {
-            msg_Err( p_this, "could not create %s", psz_mod );
-        }
-    }
-
-#else
-#   if defined( WIN32 )
-    if( mkdir( psz_dirname ) && errno != EEXIST )
-#   else
-    if( mkdir( psz_dirname, 0755 ) && errno != EEXIST )
-#   endif
+    if( utf8_mkdir( psz_dirname ) && ( errno != EEXIST ) )
     {
         msg_Err( p_this, "could not create %s (%s)",
                  psz_dirname, strerror(errno) );
+        return -1;
     }
 
-#endif
-
     return 0;
 }
 
@@ -977,68 +993,38 @@ int config_CreateDir( vlc_object_t *p_this, char *psz_dirname )
  * save.
  * Really stupid no ?
  *****************************************************************************/
-int __config_SaveConfigFile( vlc_object_t *p_this, const char *psz_module_name )
+static int SaveConfigFile( vlc_object_t *p_this, const char *psz_module_name,
+                           vlc_bool_t b_autosave )
 {
     module_t *p_parser;
     vlc_list_t *p_list;
-    module_config_t *p_item;
     FILE *file;
     char p_line[1024], *p_index2;
     int i_sizebuf = 0;
     char *p_bigbuffer, *p_index;
     vlc_bool_t b_backup;
-    char *psz_filename, *psz_homedir, *psz_configfile;
     int i_index;
 
     /* Acquire config file lock */
-    vlc_mutex_lock( &p_this->p_vlc->config_lock );
+    vlc_mutex_lock( &p_this->p_libvlc->config_lock );
 
-    psz_configfile = p_this->p_vlc->psz_configfile;
-    if( !psz_configfile || !psz_configfile )
+    if (p_this->p_libvlc->psz_configfile == NULL)
     {
-        psz_homedir = p_this->p_vlc->psz_homedir;
+        const char *psz_homedir = p_this->p_libvlc->psz_homedir;
         if( !psz_homedir )
         {
-            msg_Err( p_this, "psz_homedir is null" );
-            vlc_mutex_unlock( &p_this->p_vlc->config_lock );
+            msg_Err( p_this, "no home directory defined" );
+            vlc_mutex_unlock( &p_this->p_libvlc->config_lock );
             return -1;
         }
-        psz_filename = (char *)malloc( sizeof("/" CONFIG_DIR "/" CONFIG_FILE) +
-                                       strlen(psz_homedir) );
-
-        if( psz_filename )
-            sprintf( psz_filename, "%s/" CONFIG_DIR, psz_homedir );
 
-        if( !psz_filename )
-        {
-            msg_Err( p_this, "out of memory" );
-            vlc_mutex_unlock( &p_this->p_vlc->config_lock );
-            return -1;
-        }
-
-        config_CreateDir( p_this, psz_filename );
-
-        strcat( psz_filename, "/" CONFIG_FILE );
-    }
-    else
-    {
-        psz_filename = strdup( psz_configfile );
-        if( !psz_filename )
-        {
-            msg_Err( p_this, "out of memory" );
-            vlc_mutex_unlock( &p_this->p_vlc->config_lock );
-            return -1;
-        }
+        char dirname[strlen (psz_homedir) + sizeof (DIR_SEP CONFIG_DIR)];
+        sprintf (dirname, "%s" DIR_SEP CONFIG_DIR, psz_homedir);
+        config_CreateDir (p_this, dirname);
     }
 
-    msg_Dbg( p_this, "opening config file %s", psz_filename );
-
-    file = fopen( psz_filename, "rt" );
-    if( !file )
-    {
-        msg_Warn( p_this, "config file %s does not exist yet", psz_filename );
-    }
-    else
+    file = config_OpenConfigFile (p_this, "rt");
+    if (file != NULL)
     {
         /* look for file size */
         fseek( file, 0L, SEEK_END );
@@ -1051,8 +1037,7 @@ int __config_SaveConfigFile( vlc_object_t *p_this, const char *psz_module_name )
     {
         msg_Err( p_this, "out of memory" );
         if( file ) fclose( file );
-        free( psz_filename );
-        vlc_mutex_unlock( &p_this->p_vlc->config_lock );
+        vlc_mutex_unlock( &p_this->p_libvlc->config_lock );
         return -1;
     }
     p_bigbuffer[0] = 0;
@@ -1091,8 +1076,10 @@ int __config_SaveConfigFile( vlc_object_t *p_this, const char *psz_module_name )
                 /* we don't have this section in our list so we need to back
                  * it up */
                 *p_index2 = 0;
+#if 0
                 msg_Dbg( p_this, "backing up config for unknown module \"%s\"",
                                  &p_line[1] );
+#endif
                 *p_index2 = ']';
 
                 b_backup = 1;
@@ -1119,22 +1106,21 @@ int __config_SaveConfigFile( vlc_object_t *p_this, const char *psz_module_name )
      * Save module config in file
      */
 
-    file = fopen( psz_filename, "wt" );
+    file = config_OpenConfigFile (p_this, "wt");
     if( !file )
     {
-        msg_Warn( p_this, "could not open config file %s for writing",
-                          psz_filename );
-        free( psz_filename );
         vlc_list_release( p_list );
-        vlc_mutex_unlock( &p_this->p_vlc->config_lock );
+        vlc_mutex_unlock( &p_this->p_libvlc->config_lock );
         return -1;
     }
 
-    fprintf( file, "###\n###  " COPYRIGHT_MESSAGE "\n###\n\n" );
+    fprintf( file, "\xEF\xBB\xBF###\n###  " COPYRIGHT_MESSAGE "\n###\n\n"
+       "###\n### lines begining with a '#' character are comments\n###\n\n" );
 
     /* Look for the selected module, if NULL then save everything */
     for( i_index = 0; i_index < p_list->i_count; i_index++ )
     {
+        module_config_t *p_item, *p_end;
         p_parser = (module_t *)p_list->p_values[i_index].p_object ;
 
         if( psz_module_name && strcmp( psz_module_name,
@@ -1144,8 +1130,9 @@ int __config_SaveConfigFile( vlc_object_t *p_this, const char *psz_module_name )
         if( !p_parser->i_config_items )
             continue;
 
-        msg_Dbg( p_this, "saving config for module \"%s\"",
-                         p_parser->psz_object_name );
+        if( psz_module_name )
+            msg_Dbg( p_this, "saving config for module \"%s\"",
+                     p_parser->psz_object_name );
 
         fprintf( file, "[%s]", p_parser->psz_object_name );
         if( p_parser->psz_longname )
@@ -1153,14 +1140,36 @@ int __config_SaveConfigFile( vlc_object_t *p_this, const char *psz_module_name )
         else
             fprintf( file, "\n\n" );
 
-        for( p_item = p_parser->p_config;
-             p_item->i_type != CONFIG_HINT_END;
+        for( p_item = p_parser->p_config, p_end = p_item + p_parser->confsize;
+             p_item < p_end;
              p_item++ )
         {
-            char *psz_key;
+            char  *psz_key;
+            int   i_value = p_item->value.i;
+            float f_value = p_item->value.f;
+            const char  *psz_value = p_item->value.psz;
+
             if( p_item->i_type & CONFIG_HINT )
                 /* ignore hints */
                 continue;
+            /* Ignore deprecated options */
+            if( p_item->psz_current )
+                continue;
+            if( p_item->b_unsaveable )
+                /*obvious*/
+                continue;
+
+            if( b_autosave && !p_item->b_autosave )
+            {
+                i_value = p_item->saved.i;
+                f_value = p_item->saved.f;
+                psz_value = p_item->saved.psz;
+                if( !psz_value ) psz_value = p_item->orig.psz;
+            }
+            else
+            {
+                p_item->b_dirty = VLC_FALSE;
+            }
 
             switch( p_item->i_type )
             {
@@ -1170,46 +1179,62 @@ int __config_SaveConfigFile( vlc_object_t *p_this, const char *psz_module_name )
                     fprintf( file, "# %s (%s)\n", p_item->psz_text,
                              (p_item->i_type == CONFIG_ITEM_BOOL) ?
                              _("boolean") : _("integer") );
-                if( p_item->i_value == p_item->i_value_orig )
-                    fprintf( file, "#" );
-                fprintf( file, "%s=%i\n", p_item->psz_name, p_item->i_value );
+                if( i_value == p_item->orig.i )
+                    fputc ('#', file);
+                fprintf( file, "%s=%i\n", p_item->psz_name, i_value );
+
+                p_item->saved.i = i_value;
                 break;
+
             case CONFIG_ITEM_KEY:
                 if( p_item->psz_text )
                     fprintf( file, "# %s (%s)\n", p_item->psz_text,
                              _("key") );
-                if( p_item->i_value == p_item->i_value_orig )
-                    fprintf( file, "#" );
-                psz_key = ConfigKeyToString( p_item->i_value );
+                if( i_value == p_item->orig.i )
+                    fputc ('#', file);
+                psz_key = ConfigKeyToString( i_value );
                 fprintf( file, "%s=%s\n", p_item->psz_name,
                          psz_key ? psz_key : "" );
-                if ( psz_key ) free( psz_key );
+                freenull (psz_key);
+
+                p_item->saved.i = i_value;
                 break;
 
             case CONFIG_ITEM_FLOAT:
                 if( p_item->psz_text )
                     fprintf( file, "# %s (%s)\n", p_item->psz_text,
                              _("float") );
-                if( p_item->f_value == p_item->f_value_orig )
-                    fprintf( file, "#" );
-                fprintf( file, "%s=%f\n", p_item->psz_name,
-                         (double)p_item->f_value );
+                if( f_value == p_item->orig.f )
+                    fputc ('#', file);
+                fprintf( file, "%s=%f\n", p_item->psz_name, (double)f_value );
+
+                p_item->saved.f = f_value;
                 break;
 
             default:
                 if( p_item->psz_text )
                     fprintf( file, "# %s (%s)\n", p_item->psz_text,
                              _("string") );
-                if( (!p_item->psz_value && !p_item->psz_value_orig) ||
-                    (p_item->psz_value && p_item->psz_value_orig &&
-                     !strcmp( p_item->psz_value, p_item->psz_value_orig )) )
-                    fprintf( file, "#" );
+                if( (!psz_value && !p_item->orig.psz) ||
+                    (psz_value && p_item->orig.psz &&
+                     !strcmp( psz_value, p_item->orig.psz )) )
+                    fputc ('#', file);
                 fprintf( file, "%s=%s\n", p_item->psz_name,
-                         p_item->psz_value ? p_item->psz_value : "" );
+                         psz_value ?: "" );
+
+                if( b_autosave && !p_item->b_autosave ) break;
+
+                freenull (p_item->saved.psz);
+                if( (psz_value && p_item->orig.psz &&
+                     strcmp( psz_value, p_item->orig.psz )) ||
+                    !psz_value || !p_item->orig.psz)
+                    p_item->saved.psz = strdupnull (psz_value);
+                else
+                    p_item->saved.psz = NULL;
             }
         }
 
-        fprintf( file, "\n" );
+        fputc ('\n', file);
     }
 
     vlc_list_release( p_list );
@@ -1221,12 +1246,47 @@ int __config_SaveConfigFile( vlc_object_t *p_this, const char *psz_module_name )
     free( p_bigbuffer );
 
     fclose( file );
-    free( psz_filename );
-    vlc_mutex_unlock( &p_this->p_vlc->config_lock );
+    vlc_mutex_unlock( &p_this->p_libvlc->config_lock );
 
     return 0;
 }
 
+int config_AutoSaveConfigFile( vlc_object_t *p_this )
+{
+    vlc_list_t *p_list;
+    int i_index, i_count;
+
+    /* Check if there's anything to save */
+    vlc_mutex_lock( &p_this->p_libvlc->config_lock );
+    p_list = vlc_list_find( p_this, VLC_OBJECT_MODULE, FIND_ANYWHERE );
+    i_count = p_list->i_count;
+    for( i_index = 0; i_index < i_count; i_index++ )
+    {
+        module_t *p_parser = (module_t *)p_list->p_values[i_index].p_object ;
+        module_config_t *p_item, *p_end;
+
+        if( !p_parser->i_config_items ) continue;
+
+        for( p_item = p_parser->p_config, p_end = p_item + p_parser->confsize;
+             p_item < p_end;
+             p_item++ )
+        {
+            if( p_item->b_autosave && p_item->b_dirty ) break;
+        }
+        break;
+    }
+    vlc_list_release( p_list );
+    vlc_mutex_unlock( &p_this->p_libvlc->config_lock );
+
+    if( i_index == i_count ) return VLC_SUCCESS;
+    return SaveConfigFile( p_this, 0, VLC_TRUE );
+}
+
+int __config_SaveConfigFile( vlc_object_t *p_this, const char *psz_module_name )
+{
+    return SaveConfigFile( p_this, psz_module_name, VLC_FALSE );
+}
+
 /*****************************************************************************
  * config_LoadCmdLine: parse command line
  *****************************************************************************
@@ -1242,7 +1302,6 @@ int __config_LoadCmdLine( vlc_object_t *p_this, int *pi_argc, char *ppsz_argv[],
     int i_cmd, i_index, i_opts, i_shortopts, flag, i_verbose = 0;
     module_t *p_parser;
     vlc_list_t *p_list;
-    module_config_t *p_item;
     struct option *p_longopts;
     int i_modules_index;
 
@@ -1251,11 +1310,11 @@ int __config_LoadCmdLine( vlc_object_t *p_this, int *pi_argc, char *ppsz_argv[],
     char *psz_shortopts;
 
     /* Set default configuration and copy arguments */
-    p_this->p_vlc->i_argc    = *pi_argc;
-    p_this->p_vlc->ppsz_argv = ppsz_argv;
+    p_this->p_libvlc->i_argc    = *pi_argc;
+    p_this->p_libvlc->ppsz_argv = ppsz_argv;
 
-#ifdef SYS_DARWIN
-    /* When vlc.app is run by double clicking in Mac OS X, the 2nd arg
+#ifdef __APPLE__
+    /* When VLC.app is run by double clicking in Mac OS X, the 2nd arg
      * is the PSN - process serial number (a unique PID-ish thingie)
      * still ok for real Darwin & when run from command line */
     if ( (*pi_argc > 1) && (strncmp( ppsz_argv[ 1 ] , "-psn" , 4 ) == 0) )
@@ -1288,7 +1347,7 @@ int __config_LoadCmdLine( vlc_object_t *p_this, int *pi_argc, char *ppsz_argv[],
         p_parser = (module_t *)p_list->p_values[i_modules_index].p_object ;
 
         /* count the number of exported configuration options (to allocate
-         * longopts). We also need to allocate space for too options when
+         * longopts). We also need to allocate space for two options when
          * dealing with boolean to allow for --foo and --no-foo */
         i_opts += p_parser->i_config_items
                      + 2 * p_parser->i_bool_items;
@@ -1325,7 +1384,7 @@ int __config_LoadCmdLine( vlc_object_t *p_this, int *pi_argc, char *ppsz_argv[],
             vlc_list_release( p_list );
             return -1;
         }
-        memcpy( ppsz_argv, p_this->p_vlc->ppsz_argv,
+        memcpy( ppsz_argv, p_this->p_libvlc->ppsz_argv,
                 *pi_argc * sizeof(char *) );
     }
 
@@ -1340,13 +1399,14 @@ int __config_LoadCmdLine( vlc_object_t *p_this, int *pi_argc, char *ppsz_argv[],
     for( i_modules_index = 0; i_modules_index < p_list->i_count;
          i_modules_index++ )
     {
+        module_config_t *p_item, *p_end;
         p_parser = (module_t *)p_list->p_values[i_modules_index].p_object ;
 
         if( !p_parser->i_config_items )
             continue;
 
-        for( p_item = p_parser->p_config;
-             p_item->i_type != CONFIG_HINT_END;
+        for( p_item = p_parser->p_config, p_end = p_item + p_parser->confsize;
+             p_item < p_end;
              p_item++ )
         {
             /* Ignore hints */
@@ -1437,30 +1497,72 @@ int __config_LoadCmdLine( vlc_object_t *p_this, int *pi_argc, char *ppsz_argv[],
 
             /* Store the configuration option */
             p_conf = config_FindConfig( p_this, psz_name );
-
-            if( p_conf ) switch( p_conf->i_type )
+            if( p_conf )
             {
-            case CONFIG_ITEM_STRING:
-            case CONFIG_ITEM_FILE:
-            case CONFIG_ITEM_DIRECTORY:
-            case CONFIG_ITEM_MODULE:
-                config_PutPsz( p_this, psz_name, optarg );
-                break;
-            case CONFIG_ITEM_INTEGER:
-                config_PutInt( p_this, psz_name, strtol(optarg, 0, 0));
-                break;
-            case CONFIG_ITEM_FLOAT:
-                config_PutFloat( p_this, psz_name, (float)atof(optarg) );
-                break;
-            case CONFIG_ITEM_KEY:
-                config_PutInt( p_this, psz_name, ConfigStringToKey( optarg ) );
-                break;
-            case CONFIG_ITEM_BOOL:
-                config_PutInt( p_this, psz_name, !flag );
-                break;
-            }
+                /* Check if the option is deprecated */
+                if( p_conf->psz_current )
+                {
+                    if( !strcmp(p_conf->psz_current,"SUPPRESSED") )
+                    {
+                        if( !b_ignore_errors )
+                        {
+                            fprintf(stderr,
+                                    "Warning: option --%s is no longer used.\n",
+                                    p_conf->psz_name);
+                        }
+                       continue;
+                    }
+                    if( !b_ignore_errors )
+                    {
+                        if( p_conf->b_strict )
+                        {
+                            fprintf( stderr,
+                                     "Error: option --%s is deprecated. "
+                                     "Use --%s instead.\n",
+                                     p_conf->psz_name, p_conf->psz_current);
+                            /*free */
+                            for( i_index = 0; p_longopts[i_index].name; i_index++ )
+                                free( (char *)p_longopts[i_index].name );
+
+                            free( p_longopts );
+                            free( psz_shortopts );
+                            return -1;
+                        }
+                        fprintf(stderr,
+                                "Warning: option --%s is deprecated. "
+                                "You should use --%s instead.\n",
+                                p_conf->psz_name, p_conf->psz_current);
+                    }
+                    psz_name = p_conf->psz_current;
+                    p_conf = config_FindConfig( p_this, psz_name );
+                }
 
-            continue;
+                switch( p_conf->i_type )
+                {
+                    case CONFIG_ITEM_STRING:
+                    case CONFIG_ITEM_FILE:
+                    case CONFIG_ITEM_DIRECTORY:
+                    case CONFIG_ITEM_MODULE:
+                    case CONFIG_ITEM_MODULE_LIST:
+                    case CONFIG_ITEM_MODULE_LIST_CAT:
+                    case CONFIG_ITEM_MODULE_CAT:
+                        config_PutPsz( p_this, psz_name, optarg );
+                        break;
+                    case CONFIG_ITEM_INTEGER:
+                        config_PutInt( p_this, psz_name, strtol(optarg, 0, 0));
+                        break;
+                    case CONFIG_ITEM_FLOAT:
+                        config_PutFloat( p_this, psz_name, (float)atof(optarg) );
+                        break;
+                    case CONFIG_ITEM_KEY:
+                        config_PutInt( p_this, psz_name, ConfigStringToKey( optarg ) );
+                        break;
+                    case CONFIG_ITEM_BOOL:
+                        config_PutInt( p_this, psz_name, !flag );
+                        break;
+                }
+                continue;
+            }
         }
 
         /* A short option has been recognized */
@@ -1468,47 +1570,50 @@ int __config_LoadCmdLine( vlc_object_t *p_this, int *pi_argc, char *ppsz_argv[],
         {
             switch( pp_shortopts[i_cmd]->i_type )
             {
-            case CONFIG_ITEM_STRING:
-            case CONFIG_ITEM_FILE:
-            case CONFIG_ITEM_DIRECTORY:
-            case CONFIG_ITEM_MODULE:
-                config_PutPsz( p_this, pp_shortopts[i_cmd]->psz_name, optarg );
-                break;
-            case CONFIG_ITEM_INTEGER:
-                if( i_cmd == 'v' )
-                {
-                    if( optarg )
+                case CONFIG_ITEM_STRING:
+                case CONFIG_ITEM_FILE:
+                case CONFIG_ITEM_DIRECTORY:
+                case CONFIG_ITEM_MODULE:
+                case CONFIG_ITEM_MODULE_CAT:
+                case CONFIG_ITEM_MODULE_LIST:
+                case CONFIG_ITEM_MODULE_LIST_CAT:
+                    config_PutPsz( p_this, pp_shortopts[i_cmd]->psz_name, optarg );
+                    break;
+                case CONFIG_ITEM_INTEGER:
+                    if( i_cmd == 'v' )
                     {
-                        if( *optarg == 'v' ) /* eg. -vvv */
+                        if( optarg )
                         {
-                            i_verbose++;
-                            while( *optarg == 'v' )
+                            if( *optarg == 'v' ) /* eg. -vvv */
                             {
                                 i_verbose++;
-                                optarg++;
+                                while( *optarg == 'v' )
+                                {
+                                    i_verbose++;
+                                    optarg++;
+                                }
+                            }
+                            else
+                            {
+                                i_verbose += atoi( optarg ); /* eg. -v2 */
                             }
                         }
                         else
                         {
-                            i_verbose += atoi( optarg ); /* eg. -v2 */
+                            i_verbose++; /* -v */
                         }
+                        config_PutInt( p_this, pp_shortopts[i_cmd]->psz_name,
+                                               i_verbose );
                     }
                     else
                     {
-                        i_verbose++; /* -v */
+                        config_PutInt( p_this, pp_shortopts[i_cmd]->psz_name,
+                                               strtol(optarg, 0, 0) );
                     }
-                    config_PutInt( p_this, pp_shortopts[i_cmd]->psz_name,
-                                           i_verbose );
-                }
-                else
-                {
-                    config_PutInt( p_this, pp_shortopts[i_cmd]->psz_name,
-                                           strtol(optarg, 0, 0) );
-                }
-                break;
-            case CONFIG_ITEM_BOOL:
-                config_PutInt( p_this, pp_shortopts[i_cmd]->psz_name, 1 );
-                break;
+                    break;
+                case CONFIG_ITEM_BOOL:
+                    config_PutInt( p_this, pp_shortopts[i_cmd]->psz_name, 1 );
+                    break;
             }
 
             continue;
@@ -1517,8 +1622,9 @@ int __config_LoadCmdLine( vlc_object_t *p_this, int *pi_argc, char *ppsz_argv[],
         /* Internal error: unknown option */
         if( !b_ignore_errors )
         {
-            fprintf( stderr, "%s: unknown option ",
-                             p_this->p_vlc->psz_object_name );
+            fprintf( stderr, "%s: unknown option"
+                     " or missing mandatory argument ",
+                     p_this->p_libvlc->psz_object_name );
             if( optopt )
             {
                 fprintf( stderr, "`-%c'\n", optopt );
@@ -1528,8 +1634,10 @@ int __config_LoadCmdLine( vlc_object_t *p_this, int *pi_argc, char *ppsz_argv[],
                 fprintf( stderr, "`%s'\n", ppsz_argv[optind-1] );
             }
             fprintf( stderr, "Try `%s --help' for more information.\n",
-                             p_this->p_vlc->psz_object_name );
+                             p_this->p_libvlc->psz_object_name );
 
+            for( i_index = 0; p_longopts[i_index].name; i_index++ )
+                free( (char *)p_longopts[i_index].name );
             free( p_longopts );
             free( psz_shortopts );
             return -1;
@@ -1546,16 +1654,40 @@ int __config_LoadCmdLine( vlc_object_t *p_this, int *pi_argc, char *ppsz_argv[],
     return 0;
 }
 
+/**
+ * config_GetDataDir: find directory where shared data is installed
+ *
+ * @return a string (always succeeds).
+ */
+const char *config_GetDataDir( const vlc_object_t *p_this )
+{
+#if defined (WIN32) || defined (UNDER_CE)
+    return p_this->p_libvlc_global->psz_vlcpath;
+#elif defined(__APPLE__) || defined (SYS_BEOS)
+    static char path[PATH_MAX] = "";
+
+    if( *path == '\0' )
+    {
+        snprintf( path, sizeof( path ), "%s/share",
+                  p_this->p_libvlc_global->psz_vlcpath );
+        path[sizeof( path ) - 1] = '\0';
+    }
+    return path;
+#else
+    return DATA_PATH;
+#endif
+}
+
 /*****************************************************************************
- * config_GetHomeDir: find the user's home directory.
+ * config_GetHomeDir, config_GetUserDir: find the user's home directory.
  *****************************************************************************
  * This function will try by different ways to find the user's home path.
  * Note that this function is not reentrant, it should be called only once
  * at the beginning of main where the result will be stored for later use.
  *****************************************************************************/
-char *config_GetHomeDir( void )
+static char *GetDir( vlc_bool_t b_appdata )
 {
-    char *p_tmp, *p_homedir = NULL;
+    const char *psz_localhome = NULL;
 
 #if defined(HAVE_GETPWUID)
     struct passwd *p_pw = NULL;
@@ -1563,13 +1695,16 @@ char *config_GetHomeDir( void )
 
 #if defined(WIN32) && !defined(UNDER_CE)
     typedef HRESULT (WINAPI *SHGETFOLDERPATH)( HWND, int, HANDLE, DWORD,
-                                               LPSTR );
+                                               LPWSTR );
 #ifndef CSIDL_FLAG_CREATE
 #   define CSIDL_FLAG_CREATE 0x8000
 #endif
 #ifndef CSIDL_APPDATA
 #   define CSIDL_APPDATA 0x1A
 #endif
+#ifndef CSIDL_PROFILE
+#   define CSIDL_PROFILE 0x28
+#endif
 #ifndef SHGFP_TYPE_CURRENT
 #   define SHGFP_TYPE_CURRENT 0
 #endif
@@ -1581,72 +1716,74 @@ char *config_GetHomeDir( void )
     if( ( shfolder_dll = LoadLibrary( _T("SHFolder.dll") ) ) != NULL )
     {
         SHGetFolderPath = (void *)GetProcAddress( shfolder_dll,
-                                                  _T("SHGetFolderPathA") );
+                                                  _T("SHGetFolderPathW") );
         if ( SHGetFolderPath != NULL )
         {
-            p_homedir = (char *)malloc( MAX_PATH );
-            if( !p_homedir ) return NULL;
+            wchar_t whomedir[MAX_PATH];
 
             /* get the "Application Data" folder for the current user */
             if( S_OK == SHGetFolderPath( NULL,
-                                         CSIDL_APPDATA | CSIDL_FLAG_CREATE,
+                                         (b_appdata ? CSIDL_APPDATA :
+                                           CSIDL_PROFILE) | CSIDL_FLAG_CREATE,
                                          NULL, SHGFP_TYPE_CURRENT,
-                                         p_homedir ) )
+                                         whomedir ) )
             {
                 FreeLibrary( shfolder_dll );
-                return p_homedir;
+                return FromWide( whomedir );
             }
-            free( p_homedir );
-            p_homedir = NULL;
         }
         FreeLibrary( shfolder_dll );
     }
 
 #elif defined(UNDER_CE)
 
-    wchar_t p_whomedir[MAX_PATH];
+#ifndef CSIDL_APPDATA
+#   define CSIDL_APPDATA 0x1A
+#endif
 
-    /* get the "Application Data" folder for the current user */
-    if( SHGetSpecialFolderPath( NULL, p_whomedir, CSIDL_APPDATA, 1 ) )
-    {
-        p_homedir = (char *)malloc( MAX_PATH );
-        if( !p_homedir ) return NULL;
+    wchar_t whomedir[MAX_PATH];
 
-        sprintf( p_homedir, "%ls", p_whomedir );
-        return p_homedir;
-    }
+    /* get the "Application Data" folder for the current user */
+    if( SHGetSpecialFolderPath( NULL, whomedir, CSIDL_APPDATA, 1 ) )
+        return FromWide( whomedir );
 #endif
 
 #if defined(HAVE_GETPWUID)
     if( ( p_pw = getpwuid( getuid() ) ) == NULL )
 #endif
     {
-        if( ( p_tmp = getenv( "HOME" ) ) == NULL )
+        psz_localhome = getenv( "HOME" );
+        if( psz_localhome == NULL )
         {
-            if( ( p_tmp = getenv( "TMP" ) ) == NULL )
-            {
-                p_tmp = "/tmp";
-            }
+            psz_localhome = getenv( "TMP" );
+            if( psz_localhome == NULL )
+                psz_localhome = "/tmp";
         }
-
-        p_homedir = strdup( p_tmp );
     }
 #if defined(HAVE_GETPWUID)
     else
-    {
-        p_homedir = strdup( p_pw->pw_dir );
-    }
+        psz_localhome = p_pw->pw_dir;
 #endif
 
-    return p_homedir;
+    return FromLocaleDup( psz_localhome );
+}
+
+char *config_GetHomeDir( void )
+{
+    return GetDir( VLC_TRUE );
+}
+
+char *config_GetUserDir( void )
+{
+    return GetDir( VLC_FALSE );
 }
 
 
-static int ConfigStringToKey( char *psz_key )
+static int ConfigStringToKey( const char *psz_key )
 {
     int i_key = 0;
     unsigned int i;
-    char *psz_parser = strchr( psz_key, '-' );
+    const char *psz_parser = strchr( psz_key, '-' );
     while( psz_parser && psz_parser != psz_key )
     {
         for( i = 0; i < sizeof(vlc_modifiers) / sizeof(key_descriptor_t); i++ )