[FFmpeg-cvslog] r14612 - trunk/tests/seek.regression.ref
Benoit Fouet
benoit.fouet
Tue Aug 5 11:19:38 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
>> 2) why exactly they changed (just using the ebml parser is not really a reason)
>> 3) explain why the new values are right and why the old values were wrong.
>>
>
> 4) should be done in the same commit as the change that changes the
> test values
>
>
that should also be #1
--
Benoit Fouet
Purple Labs S.A.
www.purplelabs.com
More information about the ffmpeg-cvslog
mailing list