# include "config.h"
#endif
-#include <vlc/vlc.h>
+#include <vlc_common.h>
#include "../libvlc.h"
-#include "vlc_keys.h"
-#include "vlc_charset.h"
+#include <vlc_keys.h>
+#include <vlc_charset.h>
-#include <errno.h> /* errno */
-#include <limits.h>
-
-#ifdef HAVE_UNISTD_H
-# include <unistd.h> /* getuid() */
-#endif
-
-#ifdef HAVE_GETOPT_LONG
-# ifdef HAVE_GETOPT_H
-# include <getopt.h> /* getopt() */
-# endif
-#else
-# include "../extras/getopt.h"
-#endif
-
-#if defined(HAVE_GETPWUID)
-# include <pwd.h> /* getpwuid() */
-#endif
-
-#if defined( HAVE_SYS_STAT_H )
-# include <sys/stat.h>
-#endif
-#if defined( HAVE_SYS_TYPES_H )
-# include <sys/types.h>
-#endif
-#if defined( WIN32 )
-# if !defined( UNDER_CE )
-# include <direct.h>
-# endif
-#include <tchar.h>
-#endif
+#include "../extras/getopt.h"
#include "configuration.h"
#include "modules/modules.h"
-/*****************************************************************************
- * config_LoadCmdLine: parse command line
- *****************************************************************************
+#include <assert.h>
+
+#undef config_LoadCmdLine
+/**
* Parse command line for configuration options.
+ *
* Now that the module_bank has been initialized, we can dynamically
* generate the longopts structure used by getops. We have to do it this way
* because we don't know (and don't want to know) in advance the configuration
* options used (ie. exported) by each module.
- *****************************************************************************/
-int __config_LoadCmdLine( vlc_object_t *p_this, int *pi_argc,
- const char *ppsz_argv[],
- vlc_bool_t b_ignore_errors )
+ *
+ * @param p_this object to write command line options as variables to
+ * @param pi_argc number of command line arguments [IN/OUT]
+ * @param ppsz_args commandl ine arguments [IN/OUT]
+ * @param b_ignore_errors whether to ignore parsing errors
+ * @return 0 on success, -1 on error.
+ *
+ * @warning This function is not re-entrant (because of getopt_long()).
+ * It must be called with the module bank initialization global lock held.
+ * FIXME: this still breaks if getopt() is used outside of LibVLC.
+ */
+int config_LoadCmdLine( vlc_object_t *p_this, int *pi_argc,
+ const char *ppsz_argv[], bool b_ignore_errors )
{
int i_cmd, i_index, i_opts, i_shortopts, flag, i_verbose = 0;
module_t *p_parser;
- vlc_list_t *p_list;
struct option *p_longopts;
- int i_modules_index;
const char **argv_copy = NULL;
/* Short options */
module_config_t *pp_shortopts[256];
char *psz_shortopts;
-#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) )
- /* for example -psn_0_9306113 */
- {
- /* GDMF!... I can't do this or else the MacOSX window server will
- * not pick up the PSN and not register the app and we crash...
- * hence the following kludge otherwise we'll get confused w/ argv[1]
- * being an input file name */
-#if 0
- ppsz_argv[ 1 ] = NULL;
-#endif
- *pi_argc = *pi_argc - 1;
- pi_argc--;
- return 0;
- }
-#endif
-
/* List all modules */
- p_list = vlc_list_find( p_this, VLC_OBJECT_MODULE, FIND_ANYWHERE );
+ module_t **list = module_list_get (NULL);
/*
* Generate the longopts and shortopts structures used by getopt_long
*/
i_opts = 0;
- for( i_modules_index = 0; i_modules_index < p_list->i_count;
- i_modules_index++ )
- {
- p_parser = (module_t *)p_list->p_values[i_modules_index].p_object ;
-
+ for (size_t i = 0; (p_parser = list[i]) != NULL; i++)
/* count the number of exported configuration options (to allocate
* 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;
- }
+ i_opts += p_parser->i_config_items + 2 * p_parser->i_bool_items;
p_longopts = malloc( sizeof(struct option) * (i_opts + 1) );
if( p_longopts == NULL )
{
- msg_Err( p_this, "out of memory" );
- vlc_list_release( p_list );
+ module_list_free (list);
return -1;
}
psz_shortopts = malloc( sizeof( char ) * (2 * i_opts + 1) );
if( psz_shortopts == NULL )
{
- msg_Err( p_this, "out of memory" );
free( p_longopts );
- vlc_list_release( p_list );
+ module_list_free (list);
return -1;
}
argv_copy = (const char**)malloc( *pi_argc * sizeof(char *) );
if( argv_copy == NULL )
{
- msg_Err( p_this, "out of memory" );
free( psz_shortopts );
free( p_longopts );
- vlc_list_release( p_list );
+ module_list_free (list);
return -1;
}
memcpy( argv_copy, ppsz_argv, *pi_argc * sizeof(char *) );
/* Fill the p_longopts and psz_shortopts structures */
i_index = 0;
- for( i_modules_index = 0; i_modules_index < p_list->i_count;
- i_modules_index++ )
+ for (size_t i = 0; (p_parser = list[i]) != NULL; i++)
{
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;
p_longopts[i_index].name = strdup( p_item->psz_name );
if( p_longopts[i_index].name == NULL ) continue;
p_longopts[i_index].has_arg =
- (p_item->i_type == CONFIG_ITEM_BOOL)?
- no_argument : required_argument;
+ (p_item->i_type == CONFIG_ITEM_BOOL) ? no_argument :
+#ifndef __APPLE__
+ required_argument;
+#else
+/* It seems that required_argument is broken on Darwin.
+ * Radar ticket #6113829 */
+ optional_argument;
+#endif
p_longopts[i_index].flag = &flag;
p_longopts[i_index].val = 0;
i_index++;
}
/* We don't need the module list anymore */
- vlc_list_release( p_list );
+ module_list_free( list );
/* Close the longopts and shortopts structures */
memset( &p_longopts[i_index], 0, sizeof(struct option) );
*/
opterr = 0;
optind = 0; /* set to 0 to tell GNU getopt to reinitialize */
- while( ( i_cmd = getopt_long( *pi_argc, (char **)ppsz_argv, psz_shortopts,
+ while( ( i_cmd = vlc_getopt_long( *pi_argc, (char **)ppsz_argv, psz_shortopts,
p_longopts, &i_index ) ) != -1 )
{
/* A long option has been recognized */
if( i_cmd == 0 )
{
module_config_t *p_conf;
- char *psz_name = (char *)p_longopts[i_index].name;
+ const char *psz_name = p_longopts[i_index].name;
/* Check if we deal with a --nofoo or --no-foo long option */
if( flag ) psz_name += psz_name[2] == '-' ? 3 : 2;
psz_name = p_conf->psz_name;
}
-
+#ifdef __APPLE__
+ if( p_conf->i_type != CONFIG_ITEM_BOOL && !optarg )
+ {
+ fprintf( stderr, "Warning: missing argument for option --%s\n", p_conf->psz_name );
+ fprintf( stderr, "Try specifying options as '--optionname=value' instead of '--optionname value'\n" );
+ continue;
+ }
+#endif
switch( p_conf->i_type )
{
case CONFIG_ITEM_STRING:
case CONFIG_ITEM_MODULE_LIST:
case CONFIG_ITEM_MODULE_LIST_CAT:
case CONFIG_ITEM_MODULE_CAT:
- config_PutPsz( p_this, psz_name, optarg );
+ var_Create( p_this, psz_name, VLC_VAR_STRING );
+ var_SetString( p_this, psz_name, optarg );
break;
case CONFIG_ITEM_INTEGER:
- config_PutInt( p_this, psz_name, strtol(optarg, 0, 0));
+ var_Create( p_this, psz_name, VLC_VAR_INTEGER );
+ var_SetInteger( p_this, psz_name,
+ strtol(optarg, NULL, 0));
break;
case CONFIG_ITEM_FLOAT:
- config_PutFloat( p_this, psz_name, (float)atof(optarg) );
+ var_Create( p_this, psz_name, VLC_VAR_FLOAT );
+ var_SetFloat( p_this, psz_name, us_atof(optarg) );
break;
case CONFIG_ITEM_KEY:
- config_PutInt( p_this, psz_name, ConfigStringToKey( optarg ) );
+ var_Create( p_this, psz_name, VLC_VAR_INTEGER );
+ var_SetInteger( p_this, psz_name,
+ ConfigStringToKey( optarg ) );
break;
case CONFIG_ITEM_BOOL:
- config_PutInt( p_this, psz_name, !flag );
+ var_Create( p_this, psz_name, VLC_VAR_BOOL );
+ var_SetBool( p_this, psz_name, !flag );
break;
}
continue;
/* A short option has been recognized */
if( pp_shortopts[i_cmd] != NULL )
{
+ const char *name = pp_shortopts[i_cmd]->psz_name;
switch( pp_shortopts[i_cmd]->i_type )
{
case CONFIG_ITEM_STRING:
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 );
+ var_Create( p_this, name, VLC_VAR_STRING );
+ var_SetString( p_this, name, optarg );
break;
case CONFIG_ITEM_INTEGER:
+ var_Create( p_this, name, VLC_VAR_INTEGER );
if( i_cmd == 'v' )
{
if( optarg )
{
i_verbose++; /* -v */
}
- config_PutInt( p_this, pp_shortopts[i_cmd]->psz_name,
- i_verbose );
+ var_SetInteger( p_this, name, i_verbose );
}
else
{
- config_PutInt( p_this, pp_shortopts[i_cmd]->psz_name,
- strtol(optarg, 0, 0) );
+ var_SetInteger( p_this, name,
+ strtol(optarg, NULL, 0) );
}
break;
case CONFIG_ITEM_BOOL:
- config_PutInt( p_this, pp_shortopts[i_cmd]->psz_name, 1 );
+ var_Create( p_this, name, VLC_VAR_BOOL );
+ var_SetBool( p_this, name, true );
break;
}
/* Internal error: unknown option */
if( !b_ignore_errors )
{
- fprintf( stderr, "%s: unknown option"
- " or missing mandatory argument ",
- p_this->p_libvlc->psz_object_name );
+ fputs( "vlc: unknown option"
+ " or missing mandatory argument ", stderr );
if( optopt )
{
fprintf( stderr, "`-%c'\n", optopt );
{
fprintf( stderr, "`%s'\n", ppsz_argv[optind-1] );
}
- fprintf( stderr, "Try `%s --help' for more information.\n",
- p_this->p_libvlc->psz_object_name );
+ fputs( "Try `vlc --help' for more information.\n", stderr );
for( i_index = 0; p_longopts[i_index].name; i_index++ )
free( (char *)p_longopts[i_index].name );