Ok, firstly fixed the bug in parser-mpcmd.c where it
authorarpi <arpi@b3059339-0415-0410-9bf9-f77b7e298cf2>
Sun, 26 Jan 2003 16:11:35 +0000 (16:11 +0000)
committerarpi <arpi@b3059339-0415-0410-9bf9-f77b7e298cf2>
Sun, 26 Jan 2003 16:11:35 +0000 (16:11 +0000)
would accept a non-interger as an argument for -loop
(beginning of cmdline)..and have made it print an
error the same as when you pass a non-int to -loop
(end of cmdline) (parsed in different places).
patch by Alex Sisson <alex_sisson@yahoo.co.uk>

git-svn-id: svn://git.mplayerhq.hu/mplayer/trunk@9107 b3059339-0415-0410-9bf9-f77b7e298cf2

parser-mpcmd.c

index c407dec..1d87416 100644 (file)
@@ -216,9 +216,10 @@ m_config_parse_mp_command_line(m_config_t *config, int argc, char **argv)
          int l;
          char* end;
          l = (i+1<argc) ? strtol(argv[i+1],&end,0) : 0;
-         if(!end)
+         if(*end != '\0') {
+           mp_msg(MSGT_CFGPARSER, MSGL_ERR, "The loop option must be an integer: %s\n",argv[i+1]);
            tmp = ERR_OUT_OF_RANGE;
-         else {
+         else {
            play_tree_t* pt = last_entry ? last_entry : last_parent;
            l = l <= 0 ? -1 : l;
            pt->loop = l;