* Try to give more helpful error messages when we can't parse the
authorSteve McIntyre <steve@einval.com>
Fri, 20 Apr 2012 01:19:36 +0000 (01:19 +0000)
committerSteve McIntyre <steve@einval.com>
Fri, 20 Apr 2012 01:19:36 +0000 (01:19 +0000)
    command line, and exit rather than carry on. (Closes issue 51).

abcde
changelog

diff --git a/abcde b/abcde
index 33f7128..e9c9074 100755 (executable)
--- a/abcde
+++ b/abcde
@@ -118,7 +118,7 @@ f_seq_line ()
                done
                echo
        else
-               log error "syntax error while processing track numbers"
+               log error "syntax error while processing track numbers ($i)"
                exit 1
        fi
 }
@@ -3520,7 +3520,8 @@ else
                # Range parsing code courtesy of Vincent Ho
                # Cleaned up to use shell built-ins by Charles Steinkuehler
                if [ "${1#*[^0-9-]}" != "$1" ]; then
-                       log error "syntax error while processing track numbers"
+                       log error "syntax error while processing track numbers ($1)"
+                       exit 1
                else
                        RSTART=${1%%-*}
                        REND=${1##*-}
index 1d73e32..5b62e1f 100644 (file)
--- a/changelog
+++ b/changelog
@@ -28,6 +28,8 @@ abcde 2.5.1 UNRELEASED
   * If we don't find any CDDB or Musicbrainz information for the CD, try
     to extract CD-Text information to populate the cddbinfo template.
     (Closes issue 41).
+  * Try to give more helpful error messages when we can't parse the
+    command line, and exit rather than carry on. (Closes issue 51).
 
   [ Colin Tuckley ]
   * replace deprecated egrep with grep -E (Closes issue 24).