[FFmpeg-cvslog] ffplay: warn that -s is no longer working, suggest alternative

Stefano Sabatini git at videolan.org
Sun Jul 17 20:12:47 CEST 2011


ffmpeg | branch: master | Stefano Sabatini <stefano.sabatini-lala at poste.it> | Sat Jul 16 23:08:06 2011 +0200| [a4e540c4f58f4c3b8d2cd924d6cd7ebf8183b074] | committer: Anton Khirnov

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

Also remove now unused variables.

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

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=a4e540c4f58f4c3b8d2cd924d6cd7ebf8183b074
---

 doc/ffplay.texi |    4 ++--
 ffplay.c        |   14 +++-----------
 2 files changed, 5 insertions(+), 13 deletions(-)

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)



More information about the ffmpeg-cvslog mailing list