[FFmpeg-cvslog] r14612 - trunk/tests/seek.regression.ref

Reimar Doeffinger Reimar.Doeffinger
Tue Aug 5 08:56:00 CEST 2008


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.
If the old values were not wrong, it should explain why the change does not break
anything (e.g. some application might be using the pos values to do seeking on the
stream level without the demuxer knowing or cutting a file... - an ugly hackish
way to do things and maybe impossible with matroska, but still an example for why it matters).

Greetings,
Reimar Doeffinger




More information about the ffmpeg-cvslog mailing list