ffmpeg: complain loudly and exit in case removed -sameq option is specified
authorStefano Sabatini <stefasab@gmail.com>
Fri, 2 Nov 2012 11:57:15 +0000 (12:57 +0100)
committerStefano Sabatini <stefasab@gmail.com>
Sat, 3 Nov 2012 11:05:43 +0000 (12:05 +0100)
This is better than silently ignoring the option, which results in
unexpected quality results.

ffmpeg_opt.c

index adf40f2..8fdb4cb 100644 (file)
@@ -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)