diff options
author | Uoti Urpala <uau@mplayer2.org> | 2011-07-30 19:03:20 +0300 |
---|---|---|
committer | Uoti Urpala <uau@mplayer2.org> | 2011-07-30 19:03:20 +0300 |
commit | b33bb28ea3218399d6023a8f3fd2f9a86a3dec41 (patch) | |
tree | 203742890df05932e4f1f375330a93cc5d35937d /parser-cfg.c | |
parent | a2d74a6b200f25a721eb20a6fae4106e8275e6e5 (diff) | |
download | mpv-b33bb28ea3218399d6023a8f3fd2f9a86a3dec41.tar.bz2 mpv-b33bb28ea3218399d6023a8f3fd2f9a86a3dec41.tar.xz |
build: fix --enable-debug, remove some "#ifdef MP_DEBUG"
Recent commit 5d5ca22a6d ("options: commandline: accept --foo=xyz
style options") left some bad code under "#ifdef MP_DEBUG" in
playtree.c, which caused a compilation failure if configured with
"--enable-debug". Fix this. Having the "#ifdef MP_DEBUG" there was
completely unnecessary; it only increased the risk for this kind of
problems for no real benefit - executing the asserts under it would
have no noticeable performance or other penalty in default builds
either. Remove several cases of such harmful "#ifdef MP_DEBUG".
Diffstat (limited to 'parser-cfg.c')
-rw-r--r-- | parser-cfg.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/parser-cfg.c b/parser-cfg.c index 720b8e243f..e2c5d208e2 100644 --- a/parser-cfg.c +++ b/parser-cfg.c @@ -141,11 +141,6 @@ int m_config_parse_config_file(m_config_t *config, const char *conffile) continue; } -#ifdef MP_DEBUG - PRINT_LINENUM; - mp_msg(MSGT_CFGPARSER, MSGL_V, "option: %s\n", opt); -#endif - /* skip whitespaces */ while (isspace(line[line_pos])) ++line_pos; @@ -207,11 +202,6 @@ int m_config_parse_config_file(m_config_t *config, const char *conffile) continue; } -#ifdef MP_DEBUG - PRINT_LINENUM; - mp_msg(MSGT_CFGPARSER, MSGL_V, "parameter: %s\n", param); -#endif - /* now, check if we have some more chars on the line */ /* whitespace... */ while (isspace(line[line_pos])) |