Hi, On Thu, Oct 29, 2015 at 7:42 AM, Hendrik Leppkes <h.lepp...@gmail.com> wrote:
> On Thu, Oct 29, 2015 at 12:40 PM, Ronald S. Bultje <rsbul...@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 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel