[FFmpeg-devel] [PATCH] vp9: update timestamps in ref files using multiple invisible frames.

Ronald S. Bultje rsbultje at gmail.com
Thu Oct 29 13:28:07 CET 2015


Hi,

On Thu, Oct 29, 2015 at 7:42 AM, Hendrik Leppkes <h.leppkes at gmail.com>
wrote:

> On Thu, Oct 29, 2015 at 12:40 PM, Ronald S. Bultje <rsbultje at gmail.com>
> wrote:
> > For the 10-show-existing-frame, the source file indeed has a timestamp
> > of 3 (or 100/33) for the second visible frame, so the fix appears to
> > work correctly. For the other, only the timebase is fixed, but again
> > appears to be correct now.
> > ---
> >  tests/ref/fate/vp9-10-show-existing-frame | 2 +-
> >  tests/ref/fate/vp9-16-intra-only          | 2 +-
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/tests/ref/fate/vp9-10-show-existing-frame
> b/tests/ref/fate/vp9-10-show-existing-frame
> > index 2885b00..6a2c904 100644
> > --- a/tests/ref/fate/vp9-10-show-existing-frame
> > +++ b/tests/ref/fate/vp9-10-show-existing-frame
> > @@ -4,7 +4,7 @@
> >  #tb 0: 1/30
> >  #stream#, dts,        pts, duration,     size, hash
> >  0,          0,          0,        1,   152064,
> 18981342ec178e082519451062c3a67f
> > -0,          1,          1,        1,   152064,
> 04ab9dbeac49ec31be58f6e671698e05
> > +0,          3,          3,        1,   152064,
> 04ab9dbeac49ec31be58f6e671698e05
> >  0,          4,          4,        1,   152064,
> 4ed58a0ba93a5d97a232a50c5876cda2
> >  0,          6,          6,        1,   152064,
> a41f00034923e56ba51a0b598acc2e3a
> >  0,          7,          7,        1,   152064,
> 63fa55ae9535ccdf06d44cce8065dda6
> > diff --git a/tests/ref/fate/vp9-16-intra-only
> b/tests/ref/fate/vp9-16-intra-only
> > index 5802a53..1e8d280 100644
> > --- a/tests/ref/fate/vp9-16-intra-only
> > +++ b/tests/ref/fate/vp9-16-intra-only
> > @@ -1,7 +1,7 @@
> >  #format: frame checksums
> >  #version: 1
> >  #hash: MD5
> > -#tb 0: 12/359
> > +#tb 0: 1001/30000
> >  #stream#, dts,        pts, duration,     size, hash
> >  0,          0,          0,        1,   152064,
> d57529601178948afa4818c3c8938884
> >  0,          1,          1,        1,   152064,
> d47e00250c45733d64af067a417bcd06
> > --
> > 2.1.2
> >
>
> LGTM.


Pushed, and sorry for the breakage.

Ronald


More information about the ffmpeg-devel mailing list