From e8b3a3bd4a5b08d4fdcb7cf8c452b17a87feac28 Mon Sep 17 00:00:00 2001 From: arpi Date: Sun, 2 Feb 2003 15:06:56 +0000 Subject: print meaningful error message for missing parameter noticed by Colin Leroy git-svn-id: svn://svn.mplayerhq.hu/mplayer/trunk@9223 b3059339-0415-0410-9bf9-f77b7e298cf2 --- m_config.c | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) (limited to 'm_config.c') diff --git a/m_config.c b/m_config.c index 344285ccee..abaf2b9cf0 100644 --- a/m_config.c +++ b/m_config.c @@ -253,6 +253,10 @@ m_config_parse_option(m_config_t *config, char* arg, char* param,int set) { if(sr == M_OPT_UNKNOW){ 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; } -- cgit v1.2.3