[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