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

wm4 nfxjfg at googlemail.com
Wed Feb 3 12:52:44 CET 2016


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.


More information about the ffmpeg-devel mailing list