tweaked the --audio-quality input validation/specification
authorFilippo Valsorda <filippo.valsorda@gmail.com>
Sun, 19 Aug 2012 21:25:16 +0000 (23:25 +0200)
committerFilippo Valsorda <filippo.valsorda@gmail.com>
Sun, 19 Aug 2012 21:25:16 +0000 (23:25 +0200)
README.md
youtube-dl
youtube-dl.exe
youtube_dl/__init__.py

index 34f5bf723059e57057acbe881b53272a879d6341..8a6742d941ec5b547c1c3084a403a6bd2853d26d 100644 (file)
--- a/README.md
+++ b/README.md
@@ -91,8 +91,8 @@ which means you can modify it, redistribute it or use it however you like.
     --audio-format FORMAT    "best", "aac", "vorbis", "mp3", "m4a", or "wav";
                              best by default
     --audio-quality QUALITY  ffmpeg/avconv audio quality specification, insert a
-                             value between 0 (highest) and 9 (lowest) or a
-                             specific bitrate like 128 (default 5)
+                             value between 0 (better) and 9 (worse) for VBR or a
+                             specific bitrate like 128K (default 5)
     -k, --keep-video         keeps the video file on disk after the post-
                              processing; the video is erased by default
 
index 67f6f0c36088f5bd7ecebdb56a55865689a6709d..913619d6d342876bfe185242d7e91c8e08618447 100755 (executable)
Binary files a/youtube-dl and b/youtube-dl differ
index c55f5fa8d30a9386b9f2add1d11aaa9c470704ab..bf8ed478bec3af8f910d5974fd6e0c611a39cd94 100755 (executable)
Binary files a/youtube-dl.exe and b/youtube-dl.exe differ
index 2bc9a3fa6ec1d21175442e83d9b1514371d0414f..8817228f31ea9bd9073189a5980d2f56bbf30d44 100644 (file)
@@ -297,7 +297,7 @@ def parseOpts():
        postproc.add_option('--audio-format', metavar='FORMAT', dest='audioformat', default='best',
                        help='"best", "aac", "vorbis", "mp3", "m4a", or "wav"; best by default')
        postproc.add_option('--audio-quality', metavar='QUALITY', dest='audioquality', default='5',
-                       help='ffmpeg/avconv audio quality specification, insert a value between 0 (highest) and 9 (lowest) or a specific bitrate like 128 (default 5)')
+                       help='ffmpeg/avconv audio quality specification, insert a value between 0 (better) and 9 (worse) for VBR or a specific bitrate like 128K (default 5)')
        postproc.add_option('-k', '--keep-video', action='store_true', dest='keepvideo', default=False,
                        help='keeps the video file on disk after the post-processing; the video is erased by default')
 
@@ -443,6 +443,10 @@ def _real_main():
        if opts.extractaudio:
                if opts.audioformat not in ['best', 'aac', 'mp3', 'vorbis', 'm4a', 'wav']:
                        parser.error(u'invalid audio format specified')
+       if opts.audioquality:
+               opts.audioquality = opts.audioquality.strip('k').strip('K')
+               if not opts.audioquality.isdigit():
+                       parser.error(u'invalid audio quality specified')
 
        # File downloader
        fd = FileDownloader({