[FFmpeg-devel] [PATCH] ffmpeg: fail on experimental encoders if -strict > experimental

[Janne Grunau]

On Wed, May 26, 2010 at 08:11:42PM +0200, Michael Niedermayer wrote:
> On Wed, May 26, 2010 at 08:03:15PM +0200, Janne Grunau wrote:
> > On Wed, May 26, 2010 at 06:56:52PM +0200, Michael Niedermayer wrote:
> > > On Wed, May 26, 2010 at 09:31:55AM +0200, Janne Grunau wrote:
> > > > On Wed, May 26, 2010 at 02:08:00AM +0200, Michael Niedermayer wrote:
> > > > > On Wed, May 26, 2010 at 12:48:00AM +0200, Janne Grunau wrote:
> > > > > > @@ -3133,9 +3139,9 @@ static void opt_input_file(const char *filename)
> > > > > >  
> > > > > >      set_context_opts(ic, avformat_opts, AV_OPT_FLAG_DECODING_PARAM);
> > > > > >  
> > > > > > -    ic->video_codec_id   = find_codec_or_die(video_codec_name   , AVMEDIA_TYPE_VIDEO   , 0);
> > > > > > -    ic->audio_codec_id   = find_codec_or_die(audio_codec_name   , AVMEDIA_TYPE_AUDIO   , 0);
> > > > > > -    ic->subtitle_codec_id= find_codec_or_die(subtitle_codec_name, AVMEDIA_TYPE_SUBTITLE, 0);
> > > > > > +    ic->video_codec_id   = find_codec_or_die(video_codec_name   , AVMEDIA_TYPE_VIDEO   , 0, 0);
> > > > > > +    ic->audio_codec_id   = find_codec_or_die(audio_codec_name   , AVMEDIA_TYPE_AUDIO   , 0, 0);
> > > > > > +    ic->subtitle_codec_id= find_codec_or_die(subtitle_codec_name, AVMEDIA_TYPE_SUBTITLE, 0, 0);
> > > > > 
> > > > > 0 is not ideal, it would be nice to be able to use this for decoders as well
> > > > 
> > > > Agreed, there is unfortunately no AVCodecContext. Another hint that I'm
> > > 
> > > avcodec_opts doesnt work?
> > 
> > missed that, it works. new patch attached
> > 
> > Janne
> 
> >  ffmpeg.c |   30 +++++++++++++++++++++++-------
> >  1 file changed, 23 insertions(+), 7 deletions(-)
> > b0c437288e76eb893037f3b3a359d9291d4c391e  ffmpeg_fail_on_experimental_codecs.diff
> > commit ef854fc5ded795aebb0e02bbc0c67e44d4023987
> 
> looks ok if it works

applied

Janne