]> git.sesse.net Git - vlc/blobdiff - src/libvlc-common.c
Removes trailing spaces. Removes tabs.
[vlc] / src / libvlc-common.c
index aeed9152be10cb9cd017753d802e5356e65aa934..f97f4defc175d9745d5e9a7159a4a9932965a121 100644 (file)
  * This file contains functions to create and destroy libvlc instances
  */
 
-/*****************************************************************************
- * Pretend we are a builtin module
- *****************************************************************************/
-#define MODULE_NAME main
-#define MODULE_PATH main
-#define __BUILTIN__
-
 /*****************************************************************************
  * Preamble
  *****************************************************************************/
 #include <vlc/vlc.h>
-#include <libvlc_internal.h>
-#include <vlc/input.h>
+#include "control/libvlc_internal.h"
+#include <vlc_input.h>
+
+#include "modules/modules.h"
+#include "modules/configuration.h"
 
 #include <errno.h>                                                 /* ENOMEM */
 #include <stdio.h>                                              /* sprintf() */
@@ -67,6 +63,7 @@
 #endif
 
 #ifdef HAVE_DBUS_3
+/* used for one-instance mode */
 #   include <dbus/dbus.h>
 #endif
 
 #   include <hal/libhal.h>
 #endif
 
-#include "vlc_cpu.h"                                        /* CPU detection */
-#include "os_specific.h"
+#include "vlc_os_specific.h"
 
-#include "vlc_error.h"
+#include <vlc_playlist.h>
+#include <vlc_interface.h>
 
-#include "vlc_playlist.h"
-#include "vlc_interface.h"
+#include <vlc_aout.h>
+#include "audio_output/aout_internal.h"
 
-#include "audio_output.h"
+#include <vlc_vout.h>
 
-#include "vlc_video.h"
-#include "video_output.h"
+#include <vlc_sout.h>
+#include "stream_output/stream_output.h"
 
-#include "stream_output.h"
-#include "charset.h"
+#include <vlc_charset.h>
 
 #include "libvlc.h"
 
  * The evil global variable. We handle it with care, don't worry.
  *****************************************************************************/
 static libvlc_global_data_t   libvlc_global;
-static libvlc_global_data_t * p_libvlc_global;
-static libvlc_int_t *    p_static_vlc;
+#define p_libvlc_global (&libvlc_global)
+static libvlc_int_t *    p_static_vlc = NULL;
 static volatile unsigned int i_instances = 0;
 
 /*****************************************************************************
  * Local prototypes
  *****************************************************************************/
-void LocaleInit( vlc_object_t * );
-void LocaleDeinit( void );
+#if defined (__APPLE__) || defined (WIN32)
 static void SetLanguage   ( char const * );
+#endif
+static inline int LoadMessages (void);
 static int  GetFilenames  ( libvlc_int_t *, int, char *[] );
 static void Help          ( libvlc_int_t *, char const *psz_help_name );
 static void Usage         ( libvlc_int_t *, char const *psz_module_name );
-static void ListModules   ( libvlc_int_t * );
+static void ListModules   ( libvlc_int_t *, vlc_bool_t );
 static void Version       ( void );
 
 #ifdef WIN32
@@ -125,6 +122,11 @@ static int  VerboseCallback( vlc_object_t *, char const *,
 
 static void InitDeviceValues( libvlc_int_t * );
 
+libvlc_global_data_t *vlc_global( void )
+{
+    return &libvlc_global;
+}
+
 /*****************************************************************************
  * vlc_current_object: return the current object.
  *****************************************************************************
@@ -151,11 +153,13 @@ libvlc_int_t * libvlc_InternalCreate( void )
     int i_ret;
     libvlc_int_t * p_libvlc = NULL;
     vlc_value_t lockval;
-    char *psz_env;
+    char *psz_env = NULL;
 
+#if 0
     /* &libvlc_global never changes,
      * so we can safely call this multiple times. */
     p_libvlc_global = &libvlc_global;
+#endif
 
     /* vlc_threads_init *must* be the first internal call! No other call is
      * allowed before the thread system has been initialized. */
@@ -173,7 +177,7 @@ libvlc_int_t * libvlc_InternalCreate( void )
     if( !libvlc_global.b_ready )
     {
         /* Guess what CPU we have */
-        libvlc_global.i_cpu = CPUCapabilities();
+        cpu_flags = CPUCapabilities();
        /* The module bank will be initialized later */
         libvlc_global.p_module_bank = NULL;
 
@@ -184,8 +188,11 @@ libvlc_int_t * libvlc_InternalCreate( void )
 
     /* Allocate a libvlc instance object */
     p_libvlc = vlc_object_create( p_libvlc_global, VLC_OBJECT_LIBVLC );
-    if( p_libvlc == NULL ) { i_instances--; return NULL; }
-    p_libvlc->thread_id = 0;
+    if( p_libvlc == NULL )
+    {
+        i_instances--;
+        return NULL;
+    }
     p_libvlc->p_playlist = NULL;
     p_libvlc->psz_object_name = "libvlc";
 
@@ -211,49 +218,12 @@ libvlc_int_t * libvlc_InternalCreate( void )
     vlc_mutex_init( p_libvlc, &p_libvlc->quicktime_lock );
     vlc_thread_set_priority( p_libvlc, VLC_THREAD_PRIORITY_LOW );
 #endif
-    /* Fake attachment */
-    p_libvlc->b_attached = VLC_TRUE;
     /* Store data for the non-reentrant API */
     p_static_vlc = p_libvlc;
 
     return p_libvlc;
 }
 
-/*
- * D-Bus callbacks needed in libvlc_InternalInit()
- */
-#ifdef HAVE_DBUS_3
-/* Handling of messages received on / object */
-static DBusHandlerResult handle_root
-    ( DBusConnection *p_conn, DBusMessage *p_from, void *p_data ) 
-{
-    DBusMessage* p_msg = dbus_message_new_method_return( p_from );
-    if( !p_msg ) return DBUS_HANDLER_RESULT_NEED_MEMORY;
-
-    DBusMessageIter args;
-    dbus_message_iter_init_append( p_msg, &args );
-    dbus_uint32_t i_serial = 0;
-
-    char *p_root = malloc( strlen( "<node name='/'></node>" ) );
-    if (!p_root ) return DBUS_HANDLER_RESULT_NEED_MEMORY;
-    sprintf( p_root, "<node name='/'></node>" );
-
-    if( !dbus_message_iter_append_basic( &args, DBUS_TYPE_STRING, &p_root ) )
-            return DBUS_HANDLER_RESULT_NEED_MEMORY;
-
-    if( !dbus_connection_send( p_conn, p_msg, &i_serial ) )
-        return DBUS_HANDLER_RESULT_NEED_MEMORY;
-    dbus_connection_flush( p_conn );
-    dbus_message_unref( p_msg );
-    return DBUS_HANDLER_RESULT_HANDLED;
-}
-/* vtable passed to dbus_connection_register_object_path() */
-static DBusObjectPathVTable vlc_dbus_root_vtable = {
-    NULL, handle_root, NULL, NULL, NULL, NULL
-};
-
-#endif
-
 /**
  * Initialize a libvlc instance
  * This function initializes a previously allocated libvlc instance:
@@ -265,14 +235,14 @@ static DBusObjectPathVTable vlc_dbus_root_vtable = {
 int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
 {
     char         p_capabilities[200];
-    char *       p_tmp;
-    char *       psz_modules;
-    char *       psz_parser;
-    char *       psz_control;
+    char *       p_tmp = NULL;
+    char *       psz_modules = NULL;
+    char *       psz_parser = NULL;
+    char *       psz_control = NULL;
     vlc_bool_t   b_exit = VLC_FALSE;
     int          i_ret = VLC_EEXIT;
-    module_t    *p_help_module;
-    playlist_t  *p_playlist;
+    module_t    *p_help_module = NULL;
+    playlist_t  *p_playlist = NULL;
     vlc_value_t  val;
 #if defined( ENABLE_NLS ) \
      && ( defined( HAVE_GETTEXT ) || defined( HAVE_INCLUDED_GETTEXT ) )
@@ -302,13 +272,11 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
     /*
      * Support for gettext
      */
-    SetLanguage( "" );
-
-    /*
-     * Global iconv, must be done after setlocale()
-     * so that vlc_current_charset() works.
-     */
-    LocaleInit( (vlc_object_t *)p_libvlc );
+#ifdef HAVE_LC_MESSAGES
+    setlocale( LC_MESSAGES, "" );
+#endif
+    setlocale( LC_CTYPE, "" );
+    LoadMessages ();
 
     /* Translate "C" to the language code: "fr", "en_GB", "nl", "ru"... */
     msg_Dbg( p_libvlc, "translation test: code is \"%s\"", _("C") );
@@ -320,7 +288,7 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
     module_InitBank( p_libvlc );
 
     /* Hack: insert the help module here */
-    p_help_module = vlc_object_create( p_libvlc, VLC_OBJECT_MODULE );
+    p_help_module = vlc_module_create( VLC_OBJECT(p_libvlc) );
     if( p_help_module == NULL )
     {
         module_EndBank( p_libvlc );
@@ -328,7 +296,7 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
     }
     p_help_module->psz_object_name = "help";
     p_help_module->psz_longname = N_("Help options");
-    config_Duplicate( p_help_module, p_help_config );
+    config_Duplicate( p_help_module, libvlc_config, libvlc_config_count );
     vlc_object_attach( p_help_module, libvlc_global.p_module_bank );
     /* End hack */
 
@@ -357,22 +325,11 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
     }
 
     /* Set the config file stuff */
-    p_libvlc->psz_homedir = config_GetHomeDir();
-    p_libvlc->psz_userdir = config_GetUserDir();
-    if( p_libvlc->psz_userdir == NULL )
-        p_libvlc->psz_userdir = strdup(p_libvlc->psz_homedir);
-    p_libvlc->psz_configfile = config_GetPsz( p_libvlc, "config" );
-    if( p_libvlc->psz_configfile != NULL && p_libvlc->psz_configfile[0] == '~'
-         && p_libvlc->psz_configfile[1] == '/' )
-    {
-        char *psz = malloc( strlen(p_libvlc->psz_userdir)
-                             + strlen(p_libvlc->psz_configfile) );
-        /* This is incomplete : we should also support the ~cmassiot/ syntax. */
-        sprintf( psz, "%s/%s", p_libvlc->psz_userdir,
-                               p_libvlc->psz_configfile + 2 );
-        free( p_libvlc->psz_configfile );
-        p_libvlc->psz_configfile = psz;
-    }
+    p_libvlc->psz_homedir    = config_GetHomeDir();
+    p_libvlc->psz_configdir  = config_GetConfigDir( p_libvlc );
+    p_libvlc->psz_datadir    = config_GetUserDataDir( p_libvlc );
+    p_libvlc->psz_cachedir   = config_GetCacheDir( p_libvlc );
+    p_libvlc->psz_configfile = config_GetCustomConfigFile( p_libvlc );
 
     /* Check for plugins cache options */
     if( config_GetInt( p_libvlc, "reset-plugins-cache" ) )
@@ -392,17 +349,17 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
     if( config_GetInt( p_libvlc, "daemon" ) )
     {
 #if HAVE_DAEMON
+        char *psz_pidfile = NULL;
+
         if( daemon( 1, 0) != 0 )
         {
             msg_Err( p_libvlc, "Unable to fork vlc to daemon mode" );
             b_exit = VLC_TRUE;
         }
-
-        p_libvlc->p_libvlc_global->b_daemon = VLC_TRUE;
+        libvlc_global.b_daemon = VLC_TRUE;
 
         /* lets check if we need to write the pidfile */
-        char * psz_pidfile = config_GetPsz( p_libvlc, "pidfile" );
-
+        psz_pidfile = config_GetPsz( p_libvlc, "pidfile" );
         if( psz_pidfile != NULL )
         {
             FILE *pidfile;
@@ -446,7 +403,7 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
             close( STDOUT_FILENO );
             close( STDERR_FILENO );
 
-            p_libvlc->p_libvlc_global->b_daemon = VLC_TRUE;
+            libvlc_global.b_daemon = VLC_TRUE;
         }
 #endif
     }
@@ -529,7 +486,13 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
     /* Check for module list option */
     else if( config_GetInt( p_libvlc, "list" ) )
     {
-        ListModules( p_libvlc );
+        ListModules( p_libvlc, VLC_FALSE );
+        b_exit = VLC_TRUE;
+        i_ret = VLC_EEXITSUCCESS;
+    }
+    else if( config_GetInt( p_libvlc, "list-verbose" ) )
+    {
+        ListModules( p_libvlc, VLC_TRUE );
         b_exit = VLC_TRUE;
         i_ret = VLC_EEXITSUCCESS;
     }
@@ -608,135 +571,126 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
      */
     system_Configure( p_libvlc, &i_argc, ppsz_argv );
 
+/* FIXME: could be replaced by using Unix sockets */
 #ifdef HAVE_DBUS_3
-    /* Initialise D-Bus interface, check for other instances */
-    DBusConnection  *p_conn;
-    DBusError       dbus_error;
-    int             i_dbus_service;
-
     dbus_threads_init_default();
-    dbus_error_init( &dbus_error );
 
-    /* connect to the session bus */
-    p_conn = dbus_bus_get( DBUS_BUS_SESSION, &dbus_error );
-    if( !p_conn )
-    {
-        msg_Err( p_libvlc, "Failed to connect to the D-Bus session daemon: %s",
-                dbus_error.message );
-        dbus_error_free( &dbus_error );
-    }
-    else
+    if( config_GetInt( p_libvlc, "one-instance" ) )
     {
-        /* we request the service org.videolan.vlc */
-        i_dbus_service = dbus_bus_request_name( p_conn, VLC_DBUS_SERVICE, 0, 
-                &dbus_error );
-        if( dbus_error_is_set( &dbus_error ) )
-        { 
-            msg_Err( p_libvlc, "Error requesting %s service: %s\n",
-                    VLC_DBUS_SERVICE, dbus_error.message );
+        /* Initialise D-Bus interface, check for other instances */
+        DBusConnection  *p_conn = NULL;
+        DBusError       dbus_error;
+
+        dbus_error_init( &dbus_error );
+
+        /* connect to the session bus */
+        p_conn = dbus_bus_get( DBUS_BUS_SESSION, &dbus_error );
+        if( !p_conn )
+        {
+            msg_Err( p_libvlc, "Failed to connect to D-Bus session daemon: %s",
+                    dbus_error.message );
             dbus_error_free( &dbus_error );
         }
         else
         {
-            if( i_dbus_service != DBUS_REQUEST_NAME_REPLY_PRIMARY_OWNER )
+            /* check if VLC is available on the bus
+             * if not: D-Bus control is not enabled on the other
+             * instance and we can't pass MRLs to it */
+            DBusMessage *p_test_msg = NULL;
+            DBusMessage *p_test_reply = NULL;
+            p_test_msg =  dbus_message_new_method_call(
+                    "org.mpris.vlc", "/",
+                    "org.freedesktop.MediaPlayer", "Identity" );
+            /* block until a reply arrives */
+            p_test_reply = dbus_connection_send_with_reply_and_block(
+                    p_conn, p_test_msg, -1, &dbus_error );
+            dbus_message_unref( p_test_msg );
+            if( p_test_reply == NULL )
             {
-                if( config_GetInt( p_libvlc, "one-instance" ) )
+                dbus_error_free( &dbus_error );
+                msg_Dbg( p_libvlc, "No Media Player is running. "
+                        "Continuing normally." );
+            }
+            else
+            {
+                int i_input;
+                DBusMessage* p_dbus_msg = NULL;
+                DBusMessageIter dbus_args;
+                DBusPendingCall* p_dbus_pending = NULL;
+                dbus_bool_t b_play;
+
+                dbus_message_unref( p_test_reply );
+                msg_Warn( p_libvlc, "Another Media Player is running. Exiting");
+
+                for( i_input = optind;i_input < i_argc;i_input++ )
                 {
-                    msg_Warn( p_libvlc,
-                            "Another vlc instance exists: will now exit");
+                    msg_Dbg( p_libvlc, "Adds %s to the running Media Player",
+                            ppsz_argv[i_input] );
+
+                    p_dbus_msg = dbus_message_new_method_call(
+                            "org.mpris.vlc", "/TrackList",
+                            "org.freedesktop.MediaPlayer", "AddTrack" );
 
-                    int i_input;
-                    DBusMessage* p_dbus_msg;
-                    DBusMessageIter dbus_args;
-                    DBusPendingCall* p_dbus_pending;
-                    dbus_bool_t b_play;
+                    if ( NULL == p_dbus_msg )
+                    {
+                        msg_Err( p_libvlc, "D-Bus problem" );
+                        system_End( p_libvlc );
+                        exit( VLC_ETIMEOUT );
+                    }
 
-                    for( i_input = optind;i_input < i_argc;i_input++ )
+                    /* append MRLs */
+                    dbus_message_iter_init_append( p_dbus_msg, &dbus_args );
+                    if ( !dbus_message_iter_append_basic( &dbus_args,
+                                DBUS_TYPE_STRING, &ppsz_argv[i_input] ) )
                     {
-                        msg_Dbg( p_libvlc,
-                                "Give %s to other vlc\n",ppsz_argv[i_input] );
-
-                        p_dbus_msg = dbus_message_new_method_call(
-                                VLC_DBUS_SERVICE, VLC_DBUS_OBJECT_PATH,
-                                VLC_DBUS_INTERFACE, "AddMRL" );
-
-                        if ( NULL == p_dbus_msg )
-                        {
-                            msg_Err( p_libvlc, "D-Bus problem" );
-                            system_End( p_libvlc );
-                            exit( 0 );
-                        }
-
-                        /* append MRLs */
-                        dbus_message_iter_init_append( p_dbus_msg, &dbus_args );
-                        if ( !dbus_message_iter_append_basic( &dbus_args, 
-                                    DBUS_TYPE_STRING, &ppsz_argv[i_input] ) )
-                        {
-                            msg_Err( p_libvlc, "Out of memory" );
-                            dbus_message_unref( p_dbus_msg );
-                            system_End( p_libvlc );
-                            exit( 0 );
-                        }
-                        b_play = TRUE;
-                        if( config_GetInt( p_libvlc, "playlist-enqueue" ) )
-                            b_play = FALSE;
-                        if ( !dbus_message_iter_append_basic( &dbus_args,
-                                    DBUS_TYPE_BOOLEAN, &b_play ) )
-                        {
-                            msg_Err( p_libvlc, "Out of memory" );
-                            dbus_message_unref( p_dbus_msg );
-                            system_End( p_libvlc );
-                            exit( 0 );
-                        }
-
-                        /* send message and get a handle for a reply */
-                        if ( !dbus_connection_send_with_reply ( p_conn,
-                                    p_dbus_msg, &p_dbus_pending, -1 ) )
-                        {
-                            msg_Err( p_libvlc, "D-Bus problem" );
-                            dbus_message_unref( p_dbus_msg );
-                            system_End( p_libvlc );
-                            exit( 0 );
-                        }
-
-                        if ( NULL == p_dbus_pending )
-                        {
-                            msg_Err( p_libvlc, "D-Bus problem" );
-                            dbus_message_unref( p_dbus_msg );
-                            system_End( p_libvlc );
-                            exit( 0 );
-                        }
-                        dbus_connection_flush( p_conn );
+                        msg_Err( p_libvlc, "Out of memory" );
                         dbus_message_unref( p_dbus_msg );
-                        /* block until we receive a reply */
-                        dbus_pending_call_block( p_dbus_pending );
-                        dbus_pending_call_unref( p_dbus_pending );
+                        system_End( p_libvlc );
+                        exit( VLC_ENOMEM );
+                    }
+                    b_play = TRUE;
+                    if( config_GetInt( p_libvlc, "playlist-enqueue" ) )
+                        b_play = FALSE;
+                    if ( !dbus_message_iter_append_basic( &dbus_args,
+                                DBUS_TYPE_BOOLEAN, &b_play ) )
+                    {
+                        msg_Err( p_libvlc, "Out of memory" );
+                        dbus_message_unref( p_dbus_msg );
+                        system_End( p_libvlc );
+                        exit( VLC_ENOMEM );
                     }
 
-                    /* bye bye */
-                    system_End( p_libvlc );
-                    exit( 0 );
-                }
-                else
-                {
-                    msg_Warn( p_libvlc, 
-                            "Name %s is already owned on the session bus",
-                            VLC_DBUS_SERVICE );
-                }
-            }
-            else
-            {
-                /* register "/" object */
-                if( !dbus_connection_register_object_path( p_conn, "/", 
-                        &vlc_dbus_root_vtable, NULL ) )
-                {
-                    msg_Err( p_libvlc, "Out of memory" );
-                }
-                msg_Dbg( p_libvlc, 
-                        "We are the primary owner of %s on the session bus",
-                        VLC_DBUS_SERVICE );
+                    /* send message and get a handle for a reply */
+                    if ( !dbus_connection_send_with_reply ( p_conn,
+                                p_dbus_msg, &p_dbus_pending, -1 ) )
+                    {
+                        msg_Err( p_libvlc, "D-Bus problem" );
+                        dbus_message_unref( p_dbus_msg );
+                        system_End( p_libvlc );
+                        exit( VLC_ETIMEOUT );
+                    }
+
+                    if ( NULL == p_dbus_pending )
+                    {
+                        msg_Err( p_libvlc, "D-Bus problem" );
+                        dbus_message_unref( p_dbus_msg );
+                        system_End( p_libvlc );
+                        exit( VLC_ETIMEOUT );
+                    }
+                    dbus_connection_flush( p_conn );
+                    dbus_message_unref( p_dbus_msg );
+                    /* block until we receive a reply */
+                    dbus_pending_call_block( p_dbus_pending );
+                    dbus_pending_call_unref( p_dbus_pending );
+                } /* processes all command line MRLs */
+
+                /* bye bye */
+                system_End( p_libvlc );
+                exit( VLC_SUCCESS );
             }
         }
+        /* we unreference the connection when we've finished with it */
+        if( p_conn ) dbus_connection_unref( p_conn );
     }
 #endif
 
@@ -760,30 +714,28 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
      */
     msg_Flush( p_libvlc );
 
-    /* p_libvlc initialization. FIXME ? */
-
     if( !config_GetInt( p_libvlc, "fpu" ) )
-        libvlc_global.i_cpu &= ~CPU_CAPABILITY_FPU;
+        cpu_flags &= ~CPU_CAPABILITY_FPU;
 
 #if defined( __i386__ ) || defined( __x86_64__ )
     if( !config_GetInt( p_libvlc, "mmx" ) )
-        libvlc_global.i_cpu &= ~CPU_CAPABILITY_MMX;
+        cpu_flags &= ~CPU_CAPABILITY_MMX;
     if( !config_GetInt( p_libvlc, "3dn" ) )
-        libvlc_global.i_cpu &= ~CPU_CAPABILITY_3DNOW;
+        cpu_flags &= ~CPU_CAPABILITY_3DNOW;
     if( !config_GetInt( p_libvlc, "mmxext" ) )
-        libvlc_global.i_cpu &= ~CPU_CAPABILITY_MMXEXT;
+        cpu_flags &= ~CPU_CAPABILITY_MMXEXT;
     if( !config_GetInt( p_libvlc, "sse" ) )
-        libvlc_global.i_cpu &= ~CPU_CAPABILITY_SSE;
+        cpu_flags &= ~CPU_CAPABILITY_SSE;
     if( !config_GetInt( p_libvlc, "sse2" ) )
-        libvlc_global.i_cpu &= ~CPU_CAPABILITY_SSE2;
+        cpu_flags &= ~CPU_CAPABILITY_SSE2;
 #endif
 #if defined( __powerpc__ ) || defined( __ppc__ ) || defined( __ppc64__ )
     if( !config_GetInt( p_libvlc, "altivec" ) )
-        libvlc_global.i_cpu &= ~CPU_CAPABILITY_ALTIVEC;
+        cpu_flags &= ~CPU_CAPABILITY_ALTIVEC;
 #endif
 
 #define PRINT_CAPABILITY( capability, string )                              \
-    if( libvlc_global.i_cpu & capability )                                         \
+    if( vlc_CPU() & capability )                                            \
     {                                                                       \
         strncat( p_capabilities, string " ",                                \
                  sizeof(p_capabilities) - strlen(p_capabilities) );         \
@@ -827,9 +779,9 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
      * Initialize hotkey handling
      */
     var_Create( p_libvlc, "key-pressed", VLC_VAR_INTEGER );
-    p_libvlc->p_hotkeys = malloc( sizeof(p_hotkeys) );
+    p_libvlc->p_hotkeys = malloc( libvlc_hotkeys_size );
     /* Do a copy (we don't need to modify the strings) */
-    memcpy( p_libvlc->p_hotkeys, p_hotkeys, sizeof(p_hotkeys) );
+    memcpy( p_libvlc->p_hotkeys, libvlc_hotkeys, libvlc_hotkeys_size );
 
     /* Initialize playlist and get commandline files */
     playlist_ThreadCreate( p_libvlc );
@@ -900,6 +852,13 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
      */
     VLC_AddIntf( 0, "hotkeys,none", VLC_FALSE, VLC_FALSE );
 
+#ifdef HAVE_DBUS_3
+    /* loads dbus control interface if in one-instance mode
+     * we do it only when playlist exists, because dbus module needs it */
+    if( config_GetInt( p_libvlc, "one-instance" ) )
+        VLC_AddIntf( 0, "dbus,none", VLC_FALSE, VLC_FALSE );
+#endif
+
     /*
      * If needed, load the Xscreensaver interface
      * Currently, only for X
@@ -919,7 +878,7 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
     if( config_GetInt( p_libvlc, "syslog" ) == 1 )
     {
         const char *psz_logmode = "logmode=syslog";
-        libvlc_InternalAddIntf( 0, "logger,none", VLC_FALSE, VLC_FALSE,
+        libvlc_InternalAddIntf( p_libvlc, "logger,none", VLC_FALSE, VLC_FALSE,
                                 1, &psz_logmode );
     }
 #endif
@@ -934,6 +893,26 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
         VLC_AddIntf( 0, "netsync,none", VLC_FALSE, VLC_FALSE );
     }
 
+#ifdef WIN32
+    if( config_GetInt( p_libvlc, "prefer-system-codecs") == 1 )
+    {
+        char *psz_codecs = config_GetPsz( p_playlist, "codec" );
+        if( psz_codecs )
+        {
+            char *psz_morecodecs;
+            asprintf(&psz_morecodecs, "%s,dmo,quicktime", psz_codecs);
+            if( psz_morecodecs )
+            {
+                config_PutPsz( p_libvlc, "codec", psz_morecodecs);
+                free(psz_morecodecs);
+            }
+        }
+        else
+            config_PutPsz( p_libvlc, "codec", "dmo,quicktime");
+        free(psz_codecs);
+    }
+#endif
+
     /*
      * FIXME: kludge to use a p_libvlc-local variable for the Mozilla plugin
      */
@@ -976,10 +955,10 @@ int libvlc_InternalInit( libvlc_int_t *p_libvlc, int i_argc, char *ppsz_argv[] )
  */
 int libvlc_InternalCleanup( libvlc_int_t *p_libvlc )
 {
-    intf_thread_t      * p_intf;
-    vout_thread_t      * p_vout;
-    aout_instance_t    * p_aout;
-    announce_handler_t * p_announce;
+    intf_thread_t      * p_intf = NULL;
+    vout_thread_t      * p_vout = NULL;
+    aout_instance_t    * p_aout = NULL;
+    announce_handler_t * p_announce = NULL;
 
     /* Ask the interfaces to stop and destroy them */
     msg_Dbg( p_libvlc, "removing all interfaces" );
@@ -989,6 +968,7 @@ int libvlc_InternalCleanup( libvlc_int_t *p_libvlc )
         vlc_object_detach( p_intf );
         vlc_object_release( p_intf );
         intf_Destroy( p_intf );
+        p_intf = NULL;
     }
 
     /* Free playlist */
@@ -1040,6 +1020,29 @@ int libvlc_InternalDestroy( libvlc_int_t *p_libvlc, vlc_bool_t b_release )
 {
     vlc_value_t lockval;
 
+    if( !p_libvlc )
+        return VLC_EGENERIC;
+
+#ifndef WIN32
+    char* psz_pidfile = NULL;
+
+    if( libvlc_global.p_module_bank )
+    if( config_GetInt( p_libvlc, "daemon" ) )
+    {
+        psz_pidfile = config_GetPsz( p_libvlc, "pidfile" );
+        if( psz_pidfile != NULL )
+        {
+            msg_Dbg( p_libvlc, "removing pid file %s", psz_pidfile );
+            if( unlink( psz_pidfile ) == -1 )
+            {
+                msg_Dbg( p_libvlc, "removing pid file %s: failed: %s",
+                        psz_pidfile, strerror(errno) );
+            }
+        }
+        free ( psz_pidfile );
+    }
+#endif
+
     if( p_libvlc->p_memcpy_module )
     {
         module_Unneed( p_libvlc, p_libvlc->p_memcpy_module );
@@ -1050,7 +1053,9 @@ int libvlc_InternalDestroy( libvlc_int_t *p_libvlc, vlc_bool_t b_release )
     module_EndBank( p_libvlc );
 
     FREENULL( p_libvlc->psz_homedir );
-    FREENULL( p_libvlc->psz_userdir );
+    FREENULL( p_libvlc->psz_configdir );
+    FREENULL( p_libvlc->psz_datadir );
+    FREENULL( p_libvlc->psz_cachedir );
     FREENULL( p_libvlc->psz_configfile );
     FREENULL( p_libvlc->p_hotkeys );
 
@@ -1063,9 +1068,6 @@ int libvlc_InternalDestroy( libvlc_int_t *p_libvlc, vlc_bool_t b_release )
     {
         /* System specific cleaning code */
         system_End( p_libvlc );
-
-       /* Destroy global iconv */
-        LocaleDeinit();
     }
     vlc_mutex_unlock( lockval.p_address );
     var_Destroy( p_libvlc_global, "libvlc" );
@@ -1075,12 +1077,14 @@ int libvlc_InternalDestroy( libvlc_int_t *p_libvlc, vlc_bool_t b_release )
 
     /* Destroy mutexes */
     vlc_mutex_destroy( &p_libvlc->config_lock );
+    vlc_mutex_destroy( &p_libvlc->timer_lock );
 
     if( b_release ) vlc_object_release( p_libvlc );
     vlc_object_destroy( p_libvlc );
+    p_libvlc = NULL;
 
     /* Stop thread system: last one out please shut the door!
-     * The number of initializations of the thread system is counted, we 
+     * The number of initializations of the thread system is counted, we
      * can call this each time */
     vlc_threads_end( p_libvlc_global );
 
@@ -1096,10 +1100,13 @@ int libvlc_InternalAddIntf( libvlc_int_t *p_libvlc,
                             int i_options, const char *const *ppsz_options )
 {
     int i_err;
-    intf_thread_t *p_intf;
+    intf_thread_t *p_intf = NULL;
+
+    if( !p_libvlc )
+        return VLC_EGENERIC;
 
 #ifndef WIN32
-    if( p_libvlc->p_libvlc_global->b_daemon && b_block && !psz_module )
+    if( libvlc_global.b_daemon && b_block && !psz_module )
     {
         /* Daemon mode hack.
          * We prefer the dummy interface if none is specified. */
@@ -1112,7 +1119,6 @@ int libvlc_InternalAddIntf( libvlc_int_t *p_libvlc,
     /* Try to create the interface */
     p_intf = intf_Create( p_libvlc, psz_module ? psz_module : "$intf",
                           i_options, ppsz_options );
-
     if( p_intf == NULL )
     {
         msg_Err( p_libvlc, "interface \"%s\" initialization failed",
@@ -1132,12 +1138,13 @@ int libvlc_InternalAddIntf( libvlc_int_t *p_libvlc,
     {
         vlc_object_detach( p_intf );
         intf_Destroy( p_intf );
+        p_intf = NULL;
         return i_err;
     }
     return VLC_SUCCESS;
 };
 
-
+#if defined (__APPLE__) || defined (WIN32)
 /*****************************************************************************
  * SetLanguage: set the interface language.
  *****************************************************************************
@@ -1145,61 +1152,70 @@ int libvlc_InternalAddIntf( libvlc_int_t *p_libvlc,
  * as well as the LC_CTYPE category for string sorting and possible wide
  * character support.
  *****************************************************************************/
-static void SetLanguage ( char const *psz_lang )
+static void SetLanguage ( const char *psz_lang )
 {
-#if defined( ENABLE_NLS ) \
-     && ( defined( HAVE_GETTEXT ) || defined( HAVE_INCLUDED_GETTEXT ) )
+#ifdef __APPLE__
+    /* I need that under Darwin, please check it doesn't disturb
+     * other platforms. --Meuuh */
+    setenv( "LANG", psz_lang, 1 );
 
-    const char *          psz_path;
-#if defined( __APPLE__ ) || defined ( WIN32 ) || defined( SYS_BEOS )
-    char            psz_tmp[1024];
+#else
+    /* We set LC_ALL manually because it is the only way to set
+     * the language at runtime under eg. Windows. Beware that this
+     * makes the environment unconsistent when libvlc is unloaded and
+     * should probably be moved to a safer place like vlc.c. */
+    static char psz_lcall[20];
+    snprintf( psz_lcall, 19, "LC_ALL=%s", psz_lang );
+    psz_lcall[19] = '\0';
+    putenv( psz_lcall );
 #endif
 
-    if( psz_lang && !*psz_lang )
-    {
-#   if defined( HAVE_LC_MESSAGES )
-        setlocale( LC_MESSAGES, psz_lang );
-#   endif
-        setlocale( LC_CTYPE, psz_lang );
-    }
-    else if( psz_lang )
-    {
-#ifdef __APPLE__
-        /* I need that under Darwin, please check it doesn't disturb
-         * other platforms. --Meuuh */
-        setenv( "LANG", psz_lang, 1 );
-
-#elif defined( SYS_BEOS ) || defined( WIN32 )
-        /* We set LC_ALL manually because it is the only way to set
-         * the language at runtime under eg. Windows. Beware that this
-         * makes the environment unconsistent when libvlc is unloaded and
-         * should probably be moved to a safer place like vlc.c. */
-        static char psz_lcall[20];
-        snprintf( psz_lcall, 19, "LC_ALL=%s", psz_lang );
-        psz_lcall[19] = '\0';
-        putenv( psz_lcall );
+    setlocale( LC_ALL, psz_lang );
+}
 #endif
 
-        setlocale( LC_ALL, psz_lang );
-    }
 
+static inline int LoadMessages (void)
+{
+#if defined( ENABLE_NLS ) \
+     && ( defined( HAVE_GETTEXT ) || defined( HAVE_INCLUDED_GETTEXT ) )
     /* Specify where to find the locales for current domain */
 #if !defined( __APPLE__ ) && !defined( WIN32 ) && !defined( SYS_BEOS )
-    psz_path = LOCALEDIR;
+    static const char psz_path[] = LOCALEDIR;
 #else
-    snprintf( psz_tmp, sizeof(psz_tmp), "%s/%s", libvlc_global.psz_vlcpath,
-              "locale" );
-    psz_path = psz_tmp;
+    char psz_path[1024];
+    if (snprintf (psz_path, sizeof (psz_path), "%s/%s",
+                  libvlc_global.psz_vlcpath, "locale")
+                     >= (int)sizeof (psz_path))
+        return -1;
+
 #endif
-    if( !bindtextdomain( PACKAGE_NAME, psz_path ) )
+    if (bindtextdomain (PACKAGE_NAME, psz_path) == NULL)
     {
-        fprintf( stderr, "warning: couldn't bind domain %s in directory %s\n",
-                 PACKAGE_NAME, psz_path );
+        fprintf (stderr, "Warning: cannot bind text domain "PACKAGE_NAME
+                         " to directory %s\n", psz_path);
+        return -1;
     }
 
-    /* Set the default domain */
-    bind_textdomain_codeset( PACKAGE_NAME, "UTF-8" );
+    /* LibVLC wants all messages in UTF-8.
+     * Unfortunately, we cannot ask UTF-8 for strerror(), strsignal()
+     * and other functions that are not part of our text domain.
+     */
+    if (bind_textdomain_codeset (PACKAGE_NAME, "UTF-8") == NULL)
+    {
+        fprintf (stderr, "Error: cannot set Unicode encoding for text domain "
+                         PACKAGE_NAME"\n");
+        // Unbinds the text domain to avoid broken encoding
+        bindtextdomain (PACKAGE_NAME, "DOES_NOT_EXIST");
+        return -1;
+    }
+
+    /* LibVLC does NOT set the default textdomain, since it is a library.
+     * This could otherwise break programs using LibVLC (other than VLC).
+     * textdomain (PACKAGE_NAME);
+     */
 #endif
+    return 0;
 }
 
 /*****************************************************************************
@@ -1216,7 +1232,6 @@ static int GetFilenames( libvlc_int_t *p_vlc, int i_argc, char *ppsz_argv[] )
      * and their input options */
     for( i_opt = i_argc - 1; i_opt >= optind; i_opt-- )
     {
-        const char *psz_target;
         i_options = 0;
 
         /* Count the input options */
@@ -1228,26 +1243,11 @@ static int GetFilenames( libvlc_int_t *p_vlc, int i_argc, char *ppsz_argv[] )
 
         /* TODO: write an internal function of this one, to avoid
          *       unnecessary lookups. */
-        /* FIXME: should we convert options to UTF-8 as well ?? */
 
-#ifdef WIN32
-        if( GetVersion() < 0x80000000 )
-        {
-            VLC_AddTarget( p_vlc->i_object_id, ppsz_argv[i_opt],
-                       (char const **)( i_options ? &ppsz_argv[i_opt + 1] :
-                                        NULL ), i_options,
-                       PLAYLIST_INSERT, 0 );
-        }
-        else
-#endif
-        {
-            psz_target = FromLocale( ppsz_argv[ i_opt ] );
-            VLC_AddTarget( p_vlc->i_object_id, psz_target,
+        VLC_AddTarget( p_vlc->i_object_id, ppsz_argv[i_opt],
                        (char const **)( i_options ? &ppsz_argv[i_opt + 1] :
                                         NULL ), i_options,
                        PLAYLIST_INSERT, 0 );
-            LocaleFree( psz_target );
-        }
     }
 
     return VLC_SUCCESS;
@@ -1266,13 +1266,13 @@ static void Help( libvlc_int_t *p_this, char const *psz_help_name )
 
     if( psz_help_name && !strcmp( psz_help_name, "help" ) )
     {
-        utf8_fprintf( stdout, VLC_USAGE, p_this->psz_object_name );
+        utf8_fprintf( stdout, vlc_usage, p_this->psz_object_name );
         Usage( p_this, "help" );
         Usage( p_this, "main" );
     }
     else if( psz_help_name && !strcmp( psz_help_name, "longhelp" ) )
     {
-        utf8_fprintf( stdout, VLC_USAGE, p_this->psz_object_name );
+        utf8_fprintf( stdout, vlc_usage, p_this->psz_object_name );
         Usage( p_this, NULL );
     }
     else if( psz_help_name )
@@ -1304,6 +1304,18 @@ static void Usage( libvlc_int_t *p_this, char const *psz_module_name )
      *
      * The purpose of having bra and ket is that we might i18n them as well.
      */
+
+#   define COL(x)  "\033[" #x ";1m"
+#   define RED     COL(31)
+#   define GREEN   COL(32)
+#   define YELLOW  COL(33)
+#   define BLUE    COL(34)
+#   define MAGENTA COL(35)
+#   define CYAN    COL(36)
+#   define WHITE   COL(37)
+#   define GRAY    "\033[0m"
+#define COLOR_FORMAT_STRING (WHITE"  %s --%s"YELLOW"%s%s%s%s%s%s "GRAY)
+
 #define LINE_START 8
 #define PADDING_SPACES 25
 #ifdef WIN32
@@ -1311,40 +1323,58 @@ static void Usage( libvlc_int_t *p_this, char const *psz_module_name )
 #else
 #   define OPTION_VALUE_SEP " "
 #endif
-    vlc_list_t *p_list;
-    module_t *p_parser;
-    module_config_t *p_item;
+    vlc_list_t *p_list = NULL;
     char psz_spaces_text[PADDING_SPACES+LINE_START+1];
     char psz_spaces_longtext[LINE_START+3];
-    char psz_format[sizeof(FORMAT_STRING)];
+    char psz_format[sizeof(COLOR_FORMAT_STRING)];
     char psz_buffer[10000];
     char psz_short[4];
     int i_index;
     int i_width = ConsoleWidth() - (PADDING_SPACES+LINE_START+1);
-    vlc_bool_t b_advanced = config_GetInt( p_this, "advanced" );
-    vlc_bool_t b_description;
+    int i_width_description = i_width + PADDING_SPACES - 1;
+    vlc_bool_t b_advanced    = config_GetInt( p_this, "advanced" );
+    vlc_bool_t b_description = config_GetInt( p_this, "help-verbose" );
+    vlc_bool_t b_description_hack;
+    vlc_bool_t b_color       = config_GetInt( p_this, "color" );
 
     memset( psz_spaces_text, ' ', PADDING_SPACES+LINE_START );
     psz_spaces_text[PADDING_SPACES+LINE_START] = '\0';
     memset( psz_spaces_longtext, ' ', LINE_START+2 );
     psz_spaces_longtext[LINE_START+2] = '\0';
 
-    strcpy( psz_format, FORMAT_STRING );
+    if( b_color )
+        strcpy( psz_format, COLOR_FORMAT_STRING );
+    else
+        strcpy( psz_format, FORMAT_STRING );
 
     /* List all modules */
     p_list = vlc_list_find( p_this, VLC_OBJECT_MODULE, FIND_ANYWHERE );
 
+    /* Ugly hack to make sure that the help options always come first
+     * (part 1) */
+    if( !psz_module_name )
+        Usage( p_this, "help" );
+
     /* Enumerate the config for each module */
     for( i_index = 0; i_index < p_list->i_count; i_index++ )
     {
         vlc_bool_t b_help_module;
-
-        p_parser = (module_t *)p_list->p_values[i_index].p_object ;
+        module_t *p_parser = (module_t *)p_list->p_values[i_index].p_object;
+        module_config_t *p_item = NULL;
+        module_config_t *p_end = p_parser->p_config + p_parser->confsize;
 
         if( psz_module_name && strcmp( psz_module_name,
                                        p_parser->psz_object_name ) )
         {
-            continue;
+            const char *const *pp_shortcut = p_parser->pp_shortcuts;
+            while( *pp_shortcut )
+            {
+                if( !strcmp( psz_module_name, *pp_shortcut ) )
+                    break;
+                pp_shortcut ++;
+            }
+            if( !*pp_shortcut )
+                continue;
         }
 
         /* Ignore modules without config options */
@@ -1353,34 +1383,51 @@ static void Usage( libvlc_int_t *p_this, char const *psz_module_name )
             continue;
         }
 
+        b_help_module = !strcmp( "help", p_parser->psz_object_name );
+        /* Ugly hack to make sure that the help options always come first
+         * (part 2) */
+        if( !psz_module_name && b_help_module )
+            continue;
+
         /* Ignore modules with only advanced config options if requested */
         if( !b_advanced )
         {
             for( p_item = p_parser->p_config;
-                 p_item->i_type != CONFIG_HINT_END;
+                 p_item < p_end;
                  p_item++ )
             {
                 if( (p_item->i_type & CONFIG_ITEM) &&
                     !p_item->b_advanced ) break;
             }
-            if( p_item->i_type == CONFIG_HINT_END ) continue;
         }
 
         /* Print name of module */
         if( strcmp( "main", p_parser->psz_object_name ) )
-        utf8_fprintf( stdout, "\n %s\n", p_parser->psz_longname );
-
-        b_help_module = !strcmp( "help", p_parser->psz_object_name );
+        {
+            if( b_color )
+                utf8_fprintf( stdout, "\n " GREEN "%s" GRAY "\n",
+                              p_parser->psz_longname );
+            else
+                utf8_fprintf( stdout, "\n %s\n", p_parser->psz_longname );
+        }
+        if( p_parser->psz_help )
+        {
+            if( b_color )
+                utf8_fprintf( stdout, CYAN" %s\n"GRAY, p_parser->psz_help );
+            else
+                utf8_fprintf( stdout, " %s\n", p_parser->psz_help );
+        }
 
         /* Print module options */
         for( p_item = p_parser->p_config;
-             p_item->i_type != CONFIG_HINT_END;
+             p_item < p_end;
              p_item++ )
         {
             char *psz_text, *psz_spaces = psz_spaces_text;
             const char *psz_bra = NULL, *psz_type = NULL, *psz_ket = NULL;
             const char *psz_suf = "", *psz_prefix = NULL;
             signed int i;
+            size_t i_cur_width;
 
             /* Skip deprecated options */
             if( p_item->psz_current )
@@ -1398,7 +1445,41 @@ static void Usage( libvlc_int_t *p_this, char const *psz_module_name )
             case CONFIG_HINT_CATEGORY:
             case CONFIG_HINT_USAGE:
                 if( !strcmp( "main", p_parser->psz_object_name ) )
-                utf8_fprintf( stdout, "\n %s\n", p_item->psz_text );
+                {
+                    if( b_color )
+                        utf8_fprintf( stdout, GREEN "\n %s\n" GRAY,
+                                      p_item->psz_text );
+                    else
+                        utf8_fprintf( stdout, "\n %s\n", p_item->psz_text );
+                }
+                if( b_description && p_item->psz_longtext )
+                {
+                    if( b_color )
+                        utf8_fprintf( stdout, CYAN " %s\n" GRAY,
+                                      p_item->psz_longtext );
+                    else
+                        utf8_fprintf( stdout, " %s\n", p_item->psz_longtext );
+                }
+                break;
+
+            case CONFIG_HINT_SUBCATEGORY:
+                if( strcmp( "main", p_parser->psz_object_name ) )
+                break;
+            case CONFIG_SECTION:
+                if( b_color )
+                {
+                    utf8_fprintf( stdout, RED"   %s:\n"GRAY,
+                                  p_item->psz_text );
+                    if( b_description && p_item->psz_longtext )
+                        utf8_fprintf( stdout, MAGENTA"   %s\n"GRAY,
+                                      p_item->psz_longtext );
+                }
+                else
+                {
+                    utf8_fprintf( stdout, "   %s:\n", p_item->psz_text );
+                    if( b_description && p_item->psz_longtext )
+                        utf8_fprintf( stdout, "   %s\n", p_item->psz_longtext );
+                }
                 break;
 
             case CONFIG_ITEM_STRING:
@@ -1431,6 +1512,13 @@ static void Usage( libvlc_int_t *p_this, char const *psz_module_name )
                 psz_type = _("integer");
                 psz_ket = ">";
 
+                if( p_item->min.i || p_item->max.i )
+                {
+                    sprintf( psz_buffer, "%s [%i .. %i]", psz_type,
+                             p_item->min.i, p_item->max.i );
+                    psz_type = psz_buffer;
+                }
+
                 if( p_item->i_list )
                 {
                     psz_bra = OPTION_VALUE_SEP "{";
@@ -1450,12 +1538,18 @@ static void Usage( libvlc_int_t *p_this, char const *psz_module_name )
                 psz_bra = OPTION_VALUE_SEP "<";
                 psz_type = _("float");
                 psz_ket = ">";
+                if( p_item->min.f || p_item->max.f )
+                {
+                    sprintf( psz_buffer, "%s [%f .. %f]", psz_type,
+                             p_item->min.f, p_item->max.f );
+                    psz_type = psz_buffer;
+                }
                 break;
             case CONFIG_ITEM_BOOL:
                 psz_bra = ""; psz_type = ""; psz_ket = "";
                 if( !b_help_module )
                 {
-                    psz_suf = p_item->i_value ? _(" (default enabled)") :
+                    psz_suf = p_item->value.i ? _(" (default enabled)") :
                                                 _(" (default disabled)");
                 }
                 break;
@@ -1512,19 +1606,32 @@ static void Usage( libvlc_int_t *p_this, char const *psz_module_name )
 
             /* We wrap the rest of the output */
             sprintf( psz_buffer, "%s%s", p_item->psz_text, psz_suf );
-            b_description = config_GetInt( p_this, "help-verbose" );
+            b_description_hack = b_description;
 
  description:
             psz_text = psz_buffer;
+            i_cur_width = b_description && !b_description_hack
+                          ? i_width_description
+                          : i_width;
             while( *psz_text )
             {
                 char *psz_parser, *psz_word;
                 size_t i_end = strlen( psz_text );
 
                 /* If the remaining text fits in a line, print it. */
-                if( i_end <= (size_t)i_width )
+                if( i_end <= i_cur_width )
                 {
-                    utf8_fprintf( stdout, "%s\n", psz_text );
+                    if( b_color )
+                    {
+                        if( !b_description || b_description_hack )
+                            utf8_fprintf( stdout, BLUE"%s\n"GRAY, psz_text );
+                        else
+                            utf8_fprintf( stdout, "%s\n", psz_text );
+                    }
+                    else
+                    {
+                        utf8_fprintf( stdout, "%s\n", psz_text );
+                    }
                     break;
                 }
 
@@ -1539,7 +1646,7 @@ static void Usage( libvlc_int_t *p_this, char const *psz_module_name )
                     psz_parser = psz_parser ? psz_parser + 1
                                             : psz_text + i_end;
 
-                } while( psz_parser - psz_text <= i_width );
+                } while( (size_t)(psz_parser - psz_text) <= i_cur_width );
 
                 /* We cut a word in one of these cases:
                  *  - it's the only word in the line and it's too long.
@@ -1547,27 +1654,51 @@ static void Usage( libvlc_int_t *p_this, char const *psz_module_name )
                  *    going to wrap is longer than 40% of the width, and even
                  *    if the word would have fit in the next line. */
                 if( psz_word == psz_text
-                     || ( psz_word - psz_text < 80 * i_width / 100
-                           && psz_parser - psz_word > 40 * i_width / 100 ) )
+             || ( (size_t)(psz_word - psz_text) < 80 * i_cur_width / 100
+             && (size_t)(psz_parser - psz_word) > 40 * i_cur_width / 100 ) )
                 {
-                    char c = psz_text[i_width];
-                    psz_text[i_width] = '\0';
-                    utf8_fprintf( stdout, "%s\n%s", psz_text, psz_spaces );
-                    psz_text += i_width;
+                    char c = psz_text[i_cur_width];
+                    psz_text[i_cur_width] = '\0';
+                    if( b_color )
+                    {
+                        if( !b_description || b_description_hack )
+                            utf8_fprintf( stdout, BLUE"%s\n%s"GRAY,
+                                          psz_text, psz_spaces );
+                        else
+                            utf8_fprintf( stdout, "%s\n%s",
+                                          psz_text, psz_spaces );
+                    }
+                    else
+                    {
+                        utf8_fprintf( stdout, "%s\n%s", psz_text, psz_spaces );
+                    }
+                    psz_text += i_cur_width;
                     psz_text[0] = c;
                 }
                 else
                 {
                     psz_word[-1] = '\0';
-                    utf8_fprintf( stdout, "%s\n%s", psz_text, psz_spaces );
+                    if( b_color )
+                    {
+                        if( !b_description || b_description_hack )
+                            utf8_fprintf( stdout, BLUE"%s\n%s"GRAY,
+                                          psz_text, psz_spaces );
+                        else
+                            utf8_fprintf( stdout, "%s\n%s",
+                                          psz_text, psz_spaces );
+                    }
+                    else
+                    {
+                        utf8_fprintf( stdout, "%s\n%s", psz_text, psz_spaces );
+                    }
                     psz_text = psz_word;
                 }
             }
 
-            if( b_description && p_item->psz_longtext )
+            if( b_description_hack && p_item->psz_longtext )
             {
                 sprintf( psz_buffer, "%s%s", p_item->psz_longtext, psz_suf );
-                b_description = VLC_FALSE;
+                b_description_hack = VLC_FALSE;
                 psz_spaces = psz_spaces_longtext;
                 utf8_fprintf( stdout, "%s", psz_spaces );
                 goto description;
@@ -1585,13 +1716,15 @@ static void Usage( libvlc_int_t *p_this, char const *psz_module_name )
  * Print a list of all available modules (builtins and plugins) and a short
  * description for each one.
  *****************************************************************************/
-static void ListModules( libvlc_int_t *p_this )
+static void ListModules( libvlc_int_t *p_this, vlc_bool_t b_verbose )
 {
-    vlc_list_t *p_list;
-    module_t *p_parser;
+    vlc_list_t *p_list = NULL;
+    module_t *p_parser = NULL;
     char psz_spaces[22];
     int i_index;
 
+    vlc_bool_t b_color = config_GetInt( p_this, "color" );
+
     memset( psz_spaces, ' ', 22 );
 
 #ifdef WIN32
@@ -1614,8 +1747,52 @@ static void ListModules( libvlc_int_t *p_this )
         if( i < 0 ) i = 0;
         psz_spaces[i] = 0;
 
-        utf8_fprintf( stdout, "  %s%s %s\n", p_parser->psz_object_name,
-                         psz_spaces, p_parser->psz_longname );
+        if( b_color )
+            utf8_fprintf( stdout, GREEN"  %s%s "WHITE"%s\n"GRAY,
+                          p_parser->psz_object_name,
+                          psz_spaces,
+                          p_parser->psz_longname );
+        else
+            utf8_fprintf( stdout, "  %s%s %s\n",
+                          p_parser->psz_object_name,
+                          psz_spaces, p_parser->psz_longname );
+
+        if( b_verbose )
+        {
+            const char *const *pp_shortcut = p_parser->pp_shortcuts;
+            while( *pp_shortcut )
+            {
+                if( strcmp( *pp_shortcut, p_parser->psz_object_name ) )
+                {
+                    if( b_color )
+                        utf8_fprintf( stdout, CYAN"   s %s\n"GRAY,
+                                      *pp_shortcut );
+                    else
+                        utf8_fprintf( stdout, "   s %s\n",
+                                      *pp_shortcut );
+                }
+                pp_shortcut++;
+            }
+            if( p_parser->psz_capability )
+            {
+                if( b_color )
+                    utf8_fprintf( stdout, MAGENTA"   c %s (%d)\n"GRAY,
+                                  p_parser->psz_capability,
+                                  p_parser->i_score );
+                else
+                    utf8_fprintf( stdout, "   c %s (%d)\n",
+                                  p_parser->psz_capability,
+                                  p_parser->i_score );
+            }
+            if( p_parser->psz_program )
+            {
+                if( b_color )
+                    utf8_fprintf( stdout, YELLOW "   p %s\n"GRAY,
+                                  p_parser->psz_program );
+                else
+                    utf8_fprintf( stdout, "   p %s\n", p_parser->psz_program );
+            }
+        }
 
         psz_spaces[i] = ' ';
     }
@@ -1642,11 +1819,9 @@ static void Version( void )
     utf8_fprintf( stdout, _("Compiled by %s@%s.%s\n"),
              VLC_CompileBy(), VLC_CompileHost(), VLC_CompileDomain() );
     utf8_fprintf( stdout, _("Compiler: %s\n"), VLC_Compiler() );
-#ifndef HAVE_SHARED_LIBVLC
     if( strcmp( VLC_Changeset(), "exported" ) )
         utf8_fprintf( stdout, _("Based upon svn changeset [%s]\n"),
                  VLC_Changeset() );
-#endif
     utf8_fprintf( stdout, LICENSE_MSG );
 
 #ifdef WIN32        /* Pause the console because it's destroyed when we exit */
@@ -1663,7 +1838,7 @@ static void Version( void )
 static void ShowConsole( vlc_bool_t b_dofile )
 {
 #   ifndef UNDER_CE
-    FILE *f_help;
+    FILE *f_help = NULL;
 
     if( getenv( "PWD" ) && getenv( "PS1" ) ) return; /* cygwin shell */
 
@@ -1678,7 +1853,6 @@ static void ShowConsole( vlc_bool_t b_dofile )
         freopen( "vlc-help.txt", "wt", stdout );
         utf8_fprintf( stderr, _("\nDumped content to vlc-help.txt file.\n") );
     }
-
     else freopen( "CONOUT$", "w", stdout );
 
 #   endif
@@ -1716,8 +1890,9 @@ static int ConsoleWidth( void )
     int i_width = 80;
 
 #ifndef WIN32
-    char buf[20], *psz_parser;
-    FILE *file;
+    char buf[20];
+    char *psz_parser = NULL;
+    FILE *file = NULL;
     int i_ret;
 
     file = popen( "stty size 2>/dev/null", "r" );
@@ -1749,6 +1924,9 @@ static int VerboseCallback( vlc_object_t *p_this, const char *psz_variable,
                      vlc_value_t old_val, vlc_value_t new_val, void *param)
 {
     libvlc_int_t *p_libvlc = (libvlc_int_t *)p_this;
+    (void)psz_variable;
+    (void)old_val;
+    (void)param;
 
     if( new_val.i_int >= -1 )
     {
@@ -1765,20 +1943,21 @@ static int VerboseCallback( vlc_object_t *p_this, const char *psz_variable,
 static void InitDeviceValues( libvlc_int_t *p_vlc )
 {
 #ifdef HAVE_HAL
-    LibHalContext * ctx;
+    LibHalContext * ctx = NULL;
     int i, i_devices;
-    char **devices;
-    char *block_dev;
+    char **devices = NULL;
+    char *block_dev = NULL;
     dbus_bool_t b_dvd;
-    DBusConnection *p_connection;
-    DBusError       error;
 
 #ifdef HAVE_HAL_1
-    ctx =  libhal_ctx_new();
+    DBusConnection *p_connection = NULL;
+    DBusError       error;
+
+    ctx = libhal_ctx_new();
     if( !ctx ) return;
     dbus_error_init( &error );
     p_connection = dbus_bus_get ( DBUS_BUS_SYSTEM, &error );
-    if( dbus_error_is_set( &error ) )
+    if( dbus_error_is_set( &error ) || !p_connection )
     {
         dbus_error_free( &error );
         return;
@@ -1786,8 +1965,10 @@ static void InitDeviceValues( libvlc_int_t *p_vlc )
     libhal_ctx_set_dbus_connection( ctx, p_connection );
     if( libhal_ctx_init( ctx, &error ) )
 #else
-    if( ( ctx = hal_initialize( NULL, FALSE ) ) )
+    ctx = hal_initialize( NULL, FALSE );
+    if( ctx )
 #endif
+
     {
 #ifdef HAVE_HAL_1
         if( ( devices = libhal_get_all_devices( ctx, &i_devices, NULL ) ) )
@@ -1840,6 +2021,7 @@ static void InitDeviceValues( libvlc_int_t *p_vlc )
 
 #ifdef HAVE_HAL_1
         libhal_ctx_shutdown( ctx, NULL );
+        dbus_connection_unref( p_connection );
 #else
         hal_shutdown( ctx );
 #endif
@@ -1848,5 +2030,7 @@ static void InitDeviceValues( libvlc_int_t *p_vlc )
     {
         msg_Warn( p_vlc, "Unable to get HAL device properties" );
     }
-#endif
+#else
+    (void)p_vlc;
+#endif /* HAVE_HAL */
 }