[FFmpeg-cvslog] r14612 - trunk/tests/seek.regression.ref
Aurelien Jacobs
aurel
Tue Aug 5 19:39:20 CEST 2008
M?ns Rullg?rd wrote:
> Reimar Doeffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> writes:
>
> > On Tue, Aug 05, 2008 at 02:43:05AM +0200, aurel wrote:
> >> Author: aurel
> >> Date: Tue Aug 5 02:43:04 2008
> >> New Revision: 14612
> >>
> >> Log:
> >> matroskadec: update seek regression test to match new ebml parser
> >
> > IMO changes to regression tests should - if possible - contain
> > 1) which revision changed those values
Right. Done.
> > 2) why exactly they changed (just using the ebml parser is not really a reason)
Done.
> > 3) explain why the new values are right and why the old values were wrong.
Well, I don't think the old values where really wrong, but I think the new
ones are slightly more logical.
> 4) should be done in the same commit as the change that changes the
> test values
Indeed.
When I worked on this patchset, I did know that I would have to update some
patches with new reg test reference, and I planed to do so at the end of
my rework. Then I forgot about it. And I thought about it again, only when
I had launched my commit script.
I think applying this change in a new commit was the best option left to me.
And I did it in a hurry without a good commit message.
Sorry about this.
Aurel
More information about the ffmpeg-cvslog
mailing list