Remove no-op -q option
authorSteve McIntyre <steve@einval.com>
Sat, 25 Oct 2014 19:10:41 +0000 (20:10 +0100)
committerSteve McIntyre <steve@einval.com>
Sat, 25 Oct 2014 19:10:41 +0000 (20:10 +0100)
Closes issue 106. Thanks to vskytta for the patch

abcde
abcde.1
changelog

diff --git a/abcde b/abcde
index a89387d..4a45f27 100755 (executable)
--- a/abcde
+++ b/abcde
@@ -54,8 +54,6 @@ echo "-o <type1[,type2]...>"
 echo "       Output file type(s) (vorbis,mp3,flac,spx,mpc,wav,m4a,opus). Defaults to vorbis"
 echo "-p     Pad track numbers with 0's (if less than 10 tracks)"
 echo "-P     Use UNIX pipes to read+encode without wav files"
-echo "-q <level>"
-echo "       Set quality level (high,medium,low)"
 echo "-r <host1[,host2]...>"
 echo "       Also encode on these remote hosts"
 echo "-s <field>"
@@ -3356,12 +3354,6 @@ DEFAULT_OUTPUT_BINARIES=vorbis:oggenc,flac:flac,mp3:toolame,mp3:lame,mp3:bladeen
 # List of preferred cdromreaders - by default, run whichever we have in the path
 DEFAULT_CDROMREADERS="cdparanoia icedax cdda2wav pird"
 
-# List of quality levels associated with the encoders:
-DEFAULT_QUALITY_XLO="oggenc:-q -1,lame:-q 9,speex:--quality 1,m4a:"
-DEFAULT_QUALITY_LO="oggenc:-q 1,lame:-q 7,speex:--quality 5,m4a:"
-DEFAULT_QUALITY_HI="oggenc:-q 7,lame:--preset standard,speex:--quality 9,m4a:"
-DEFAULT_QUALITY_XHI="oggenc:-q 10,lame:--preset extreme,speex:--quality 10,m4a:"
-
 # Assume fetch if under FreeBSD. curl is used for Mac OS X. wget is used for
 # Linux/OpenBSD. ftp is user for NetBSD.
 # Let's use these checkings to determine the OS flavour, which will be used
@@ -3467,7 +3459,7 @@ fi
 
 # Parse command line options
 #while getopts 1a:bc:C:d:Dehj:klLmMnNo:pPq:r:Rs:S:t:T:vVxw:W: opt ; do
-while getopts 1a:bBc:C:d:Defghj:klLmMnNo:pPq:r:s:S:t:T:UvVxX:w:W:z opt ; do
+while getopts 1a:bBc:C:d:Defghj:klLmMnNo:pPr:s:S:t:T:UvVxX:w:W:z opt ; do
        case "$opt" in
                1) ONETRACK=y ;;
                a) ACTIONS="$OPTARG" ;;
@@ -3495,7 +3487,6 @@ while getopts 1a:bBc:C:d:Defghj:klLmMnNo:pPq:r:s:S:t:T:UvVxX:w:W:z opt ; do
                o) OUTPUTTYPE="$OPTARG" ;;
                p) PADTRACKS=y ;;
                P) USEPIPES=y ;;
-               q) QUALITYLEVEL="$OPTARG" ;;
                r) REMOTEHOSTS="$OPTARG" ;;
                R) CDDBLOCALRECURSIVE=y ;;
                s) SHOWCDDBFIELDS="$OPTARG" ;;
diff --git a/abcde.1 b/abcde.1
index 522254b..6386012 100644 (file)
--- a/abcde.1
+++ b/abcde.1
@@ -148,11 +148,6 @@ Pads track numbers with 0\'s.
 Use Unix PIPES to read and encode in one step. It disables multiple encodings,
 since the WAV audio file is never stored in the disc.
 .TP
-.B \-q [quality]
-Use a pre-defined quality level for the encoding process. Defined as low,
-medium or high. The "medium" value is used to reset the default values if other
-are used in the config file(s).
-.TP
 .B \-r [hosts...]
 Remote encode on this comma-delimited list of machines using \fBdistmp3\fR. See
 the REMOTEHOSTS configuration variable.
index 8ab1584..cfcc551 100644 (file)
--- a/changelog
+++ b/changelog
@@ -16,8 +16,10 @@ abcde 2.5.5 UNRELEASED
     patch. Closes issue 70.
   * Fix typo: s/CUEWAVEFILE/CUEWAVFILE/ so that abcde.mkcue should now
     produce working cue files. Closes issue 78.
+  * Remove no-op -q option. Closes issue 106. Thanks to vskytta for the
+    patch.
 
- -- Steve McIntyre <93sam@debian.org>  Tue, 27 Nov 2012 17:34:55 +0000
+ -- Steve McIntyre <93sam@debian.org>  Sat, 25 Oct 2014 12:11:16 -0700
 
 abcde 2.5.4