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

Michael Niedermayer michaelni
Thu Sep 9 22:32:34 CEST 2010


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


[...]
> @@ -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

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

He who knows, does not speak. He who speaks, does not know. -- Lao Tsu
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100909/2dc645cc/attachment.pgp>



More information about the ffmpeg-devel mailing list