On date Saturday 2011-07-16 23:08:18 +0200, Stefano Sabatini encoded:
> Also remove now unused variable.
> ---
>  doc/ffplay.texi |    3 ++-
>  ffplay.c        |    6 +++---
>  2 files changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/doc/ffplay.texi b/doc/ffplay.texi
> index 8e40c8f..d608526 100644
> --- a/doc/ffplay.texi
> +++ b/doc/ffplay.texi
> @@ -69,7 +69,8 @@ also sources and sinks).
>  @section Advanced options
>  @table @option
>  @item -pix_fmt @var{format}
> -Set pixel format.
> +This option has been removed. Use private options for specifying the
> +input pixel format.
>  @item -stats
>  Show the stream duration, the codec parameters, the current position in
>  the stream and the audio/video synchronisation drift.
> diff --git a/ffplay.c b/ffplay.c
> index 35c40b5..ef45804 100644
> --- a/ffplay.c
> +++ b/ffplay.c
> @@ -226,7 +226,6 @@ static int fs_screen_width;
>  static int fs_screen_height;
>  static int screen_width = 0;
>  static int screen_height = 0;
> -static enum PixelFormat frame_pix_fmt = PIX_FMT_NONE;
>  static int audio_disable;
>  static int video_disable;
>  static int wanted_stream[AVMEDIA_TYPE_NB]={
> @@ -2850,8 +2849,9 @@ static int opt_format(const char *opt, const char *arg)
>  
>  static int opt_frame_pix_fmt(const char *opt, const char *arg)
>  {
> -    frame_pix_fmt = av_get_pix_fmt(arg);
> -    return 0;
> +    av_log(NULL, AV_LOG_ERROR,
> +           "Option '%s' has been removed, use private format options 
> instead\n");
> +    return AVERROR(EINVAL);

It was crashing, fixed.
>From f048b8bee2f2741dd061140730fa30cf860be3e9 Mon Sep 17 00:00:00 2001
From: Stefano Sabatini <[email protected]>
Date: Sat, 16 Jul 2011 22:43:11 +0200
Subject: [PATCH] ffplay: warn that -pix_fmt is no longer working, suggest alternative

Also remove now unused variable.
---
 doc/ffplay.texi |    3 ++-
 ffplay.c        |    6 +++---
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/doc/ffplay.texi b/doc/ffplay.texi
index 8e40c8f..d608526 100644
--- a/doc/ffplay.texi
+++ b/doc/ffplay.texi
@@ -69,7 +69,8 @@ also sources and sinks).
 @section Advanced options
 @table @option
 @item -pix_fmt @var{format}
-Set pixel format.
+This option has been removed. Use private options for specifying the
+input pixel format.
 @item -stats
 Show the stream duration, the codec parameters, the current position in
 the stream and the audio/video synchronisation drift.
diff --git a/ffplay.c b/ffplay.c
index 35c40b5..77c9d4b 100644
--- a/ffplay.c
+++ b/ffplay.c
@@ -226,7 +226,6 @@ static int fs_screen_width;
 static int fs_screen_height;
 static int screen_width = 0;
 static int screen_height = 0;
-static enum PixelFormat frame_pix_fmt = PIX_FMT_NONE;
 static int audio_disable;
 static int video_disable;
 static int wanted_stream[AVMEDIA_TYPE_NB]={
@@ -2850,8 +2849,9 @@ static int opt_format(const char *opt, const char *arg)
 
 static int opt_frame_pix_fmt(const char *opt, const char *arg)
 {
-    frame_pix_fmt = av_get_pix_fmt(arg);
-    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_sync(const char *opt, const char *arg)
-- 
1.7.2.5

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to