[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