diff options
| author | arpi <arpi@b3059339-0415-0410-9bf9-f77b7e298cf2> | 2003-02-02 15:06:56 +0000 |
|---|---|---|
| committer | arpi <arpi@b3059339-0415-0410-9bf9-f77b7e298cf2> | 2003-02-02 15:06:56 +0000 |
| commit | e8b3a3bd4a5b08d4fdcb7cf8c452b17a87feac28 (patch) | |
| tree | 388ea21bae4c9776de946805e89c172686ae741a /m_config.c | |
| parent | 534d1a275c9cbb6515bbf8f2e65ae69de40e5219 (diff) | |
| download | mpv-e8b3a3bd4a5b08d4fdcb7cf8c452b17a87feac28.tar.bz2 mpv-e8b3a3bd4a5b08d4fdcb7cf8c452b17a87feac28.tar.xz | |
print meaningful error message for missing parameter
noticed by Colin Leroy <colin@colino.net>
git-svn-id: svn://svn.mplayerhq.hu/mplayer/trunk@9223 b3059339-0415-0410-9bf9-f77b7e298cf2
Diffstat (limited to 'm_config.c')
| -rw-r--r-- | m_config.c | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/m_config.c b/m_config.c index 344285ccee..abaf2b9cf0 100644 --- a/m_config.c +++ b/m_config.c @@ -254,6 +254,10 @@ m_config_parse_option(m_config_t *config, char* arg, char* param,int set) { mp_msg(MSGT_CFGPARSER, MSGL_ERR,"Error: option '%s' has no suboption '%s'\n",co->name,lst[2*i]); r = M_OPT_INVALID; } else + if(sr == M_OPT_MISSING_PARAM){ + mp_msg(MSGT_CFGPARSER, MSGL_ERR,"Error: suboption '%s' of '%s' must have a parameter!\n",lst[2*i],co->name); + r = M_OPT_INVALID; + } else r = sr; } } @@ -284,8 +288,14 @@ m_config_set_option(m_config_t *config, char* arg, char* param) { int m_config_check_option(m_config_t *config, char* arg, char* param) { + int r; mp_msg(MSGT_CFGPARSER, MSGL_DBG2,"Checking %s=%s\n",arg,param); - return m_config_parse_option(config,arg,param,0); + r=m_config_parse_option(config,arg,param,0); + if(r==M_OPT_MISSING_PARAM){ + mp_msg(MSGT_CFGPARSER, MSGL_ERR,"Error: option '%s' must have a parameter!\n",arg); + return M_OPT_INVALID; + } + return r; } |