[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