[FFmpeg-devel] [PATCH] avformat/dca_parser: set actual samplerate

Hendrik Leppkes h.leppkes at gmail.com
Thu Feb 4 12:19:55 CET 2016


On Wed, Feb 3, 2016 at 12:52 PM, wm4 <nfxjfg at googlemail.com> wrote:
> On Wed, 3 Feb 2016 12:45:39 +0100
> Hendrik Leppkes <h.leppkes at gmail.com> wrote:
>
>> On Wed, Feb 3, 2016 at 12:41 PM, wm4 <nfxjfg at googlemail.com> wrote:
>> > Fixes timestamps with raw .dts demuxing.
>> > ---
>> >  libavcodec/dca_parser.c | 1 +
>> >  1 file changed, 1 insertion(+)
>> >
>> > diff --git a/libavcodec/dca_parser.c b/libavcodec/dca_parser.c
>> > index 70e64a8..337a99d 100644
>> > --- a/libavcodec/dca_parser.c
>> > +++ b/libavcodec/dca_parser.c
>> > @@ -166,6 +166,7 @@ static int dca_parse(AVCodecParserContext *s, AVCodecContext *avctx,
>> >      /* read the duration and sample rate from the frame header */
>> >      if (!dca_parse_params(buf, buf_size, &duration, &sample_rate, &pc1->framesize)) {
>> >          s->duration        = duration;
>> > +        avctx->sample_rate = sample_rate;
>> >      } else
>> >          s->duration = 0;
>> >
>> > --
>> > 2.7.0
>> >
>>
>> This is incorrect.
>> http://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=00ae5b401b24592a9f7019baada5b349152ee2fc
>
> Nice one.
>
>> Alternative patch attached.
>
> Fine with me, although it would indeed be better to have the parser
> parse the extensions correctly. But until someone goes through the
> effort, this patch is better.

Applied this variant with appropriate FATE changes.


More information about the ffmpeg-devel mailing list