On Sat, 16 Jul 2011 23:13:58 +0200, Stefano Sabatini <[email protected]> wrote: Non-text part: multipart/mixed > 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 >
Ok -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
