From: Stefano Sabatini <stefano.sabatini-lala@poste.it>
Date: Sat, 16 Jul 2011 21:08:06 +0000 (+0200)
Subject: ffplay: warn that -s is no longer working, suggest alternative
X-Git-Tag: v0.8b1~1977
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a4e540c4f58f4c3b8d2cd924d6cd7ebf8183b074;p=platform%2Fupstream%2Flibav.git

ffplay: warn that -s is no longer working, suggest alternative

Also remove now unused variables.

Signed-off-by: Anton Khirnov <anton@khirnov.net>
---

diff --git a/doc/ffplay.texi b/doc/ffplay.texi
index e5de3d2..8e40c8f 100644
--- a/doc/ffplay.texi
+++ b/doc/ffplay.texi
@@ -38,8 +38,8 @@ Force displayed width.
 @item -y @var{height}
 Force displayed height.
 @item -s @var{size}
-Set frame size (WxH or abbreviation), needed for videos which don't
-contain a header with the frame size like raw YUV.
+This option has been removed. Use private format options for specifying the
+input video size.
 @item -an
 Disable audio.
 @item -vn
diff --git a/ffplay.c b/ffplay.c
index 878ae83..35c40b5 100644
--- a/ffplay.c
+++ b/ffplay.c
@@ -226,8 +226,6 @@ static int fs_screen_width;
 static int fs_screen_height;
 static int screen_width = 0;
 static int screen_height = 0;
-static int frame_width = 0;
-static int frame_height = 0;
 static enum PixelFormat frame_pix_fmt = PIX_FMT_NONE;
 static int audio_disable;
 static int video_disable;
@@ -2823,15 +2821,9 @@ static void event_loop(void)
 
 static int opt_frame_size(const char *opt, const char *arg)
 {
-    if (av_parse_video_size(&frame_width, &frame_height, arg) < 0) {
-        fprintf(stderr, "Incorrect frame size\n");
-        return AVERROR(EINVAL);
-    }
-    if ((frame_width % 2) != 0 || (frame_height % 2) != 0) {
-        fprintf(stderr, "Frame size must be a multiple of 2\n");
-        return AVERROR(EINVAL);
-    }
-    return 0;
+    av_log(NULL, AV_LOG_ERROR,
+           "Option '%s' has been removed, use private format options instead\n", opt);
+    return AVERROR(EINVAL);
 }
 
 static int opt_width(const char *opt, const char *arg)