diff options
author | Stefano Sabatini <stefasab@gmail.com> | 2012-11-02 12:57:15 +0100 |
---|---|---|
committer | Stefano Sabatini <stefasab@gmail.com> | 2012-11-03 12:05:43 +0100 |
commit | a62242678ff96eade59960d1bbf65e4f3f03344f (patch) | |
tree | 0887dea6d708ff5df3761175d351222730054ef7 | |
parent | c9eaa98a3e0b1db75680f9ea28e01f21cada1dc3 (diff) | |
download | ffmpeg-a62242678ff96eade59960d1bbf65e4f3f03344f.tar.gz |
ffmpeg: complain loudly and exit in case removed -sameq option is specified
This is better than silently ignoring the option, which results in
unexpected quality results.
-rw-r--r-- | ffmpeg_opt.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c index adf40f2b32..8fdb4cb259 100644 --- a/ffmpeg_opt.c +++ b/ffmpeg_opt.c @@ -159,8 +159,11 @@ static int opt_pad(void *optctx, const char *opt, const char *arg) static int opt_sameq(void *optctx, const char *opt, const char *arg) { - av_log(NULL, AV_LOG_WARNING, "Ignoring option '%s'\n", opt); - return 0; + av_log(NULL, AV_LOG_ERROR, "Option '%s' was removed. " + "If you are looking for an option to preserve the quality (which is not " + "what -%s was for), use -qscale 0 or an equivalent quality factor option.\n", + opt, opt); + return AVERROR(EINVAL); } static int opt_video_channel(void *optctx, const char *opt, const char *arg) |