From a4e540c4f58f4c3b8d2cd924d6cd7ebf8183b074 Mon Sep 17 00:00:00 2001 From: Stefano Sabatini Date: Sat, 16 Jul 2011 23:08:06 +0200 Subject: [PATCH] ffplay: warn that -s is no longer working, suggest alternative Also remove now unused variables. Signed-off-by: Anton Khirnov --- 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) -- 2.7.4