[FFmpeg-devel] [PATCH] G.722 decoder

Martin Storsjö martin
Thu Sep 9 23:00:19 CEST 2010


On Thu, 9 Sep 2010, Michael Niedermayer wrote:

> On Thu, Sep 09, 2010 at 10:36:41PM +0300, Martin Storsj? wrote:
> > On Wed, 8 Sep 2010, Michael Niedermayer wrote:
> > 
> > > lowres could be checked for this
> > > except that iam fine with the patch
> > 
> > Applied the decoder and demuxer/muxer. I noticed that a separate AVOption 
> > (which I named -audio_lowres) is needed for setting lowres for audio 
> 
> -alowres should work without this "a..." should be routed to the audio
> context

Oh, nifty, didn't know about that!

> > @@ -3241,6 +3241,8 @@ static void opt_input_file(const char *filename)
> >              input_codecs[nb_icodecs++] = avcodec_find_decoder_by_name(audio_codec_name);
> >              if(audio_disable)
> >                  st->discard= AVDISCARD_ALL;
> > +            if (dec->lowres)
> > +                audio_sample_rate >>= dec->lowres;
> >              break;
> >          case AVMEDIA_TYPE_VIDEO:
> >              set_context_opts(dec, avcodec_opts[AVMEDIA_TYPE_VIDEO], AV_OPT_FLAG_VIDEO_PARAM | AV_OPT_FLAG_DECODING_PARAM);
> 
> with a comment that says av_find_stream_info() can add streams and we have no
> chance to set lowres up for them early enough ...
> this hunk should be ok

Applied this hunk.

The rest of it attached again.

// Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Allow-the-lowres-option-to-affect-audio-codecs-too.patch
Type: text/x-diff
Size: 2101 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100910/a66716d0/attachment.patch>



More information about the ffmpeg-devel mailing list