[FFmpeg-cvslog] r14956 - trunk/libavformat/matroskadec.c

[Mike Melanson]

aurel wrote:
> Author: aurel
> Date: Mon Aug 25 01:57:29 2008
> New Revision: 14956
> 
> Log:
> matroskadec: don't try to seek to negative timestamp
> matroska timestamps are unsigned
> 
> 
> Modified:
>    trunk/libavformat/matroskadec.c

Either 14955 or 14956 (both from you) broke the seek regressions.
Representative breakage:


diff -u -w "/home/melanson/fate/source"/tests/libav.regression.ref
tests/data/la
vf.regression
"/home/melanson/fate/source"/tests/seek_test.sh
"/home/melanson/fate/source"/tes
ts/seek.regression.ref
--- /home/melanson/fate/source/tests/seek.regression.ref
2008-08-04 18:01
:17.000000000 -0700
+++ tests/data/seek.regression  2008-08-25 07:28:02.000000000 -0700
@@ -2962,25 +2962,25 @@
 ret: 0 st: 0 ts:0.788000 flags:0
 ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:478 size:28118 flags:1
 ret: 0 st: 0 ts:-0.317000 flags:1
-ret: 0 st: 1 dts:0.000000 pts:0.000000 pos:28603 size:208 flags:1
+ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:478 size:28118 flags:1
 ret: 0 st: 1 ts:2.577000 flags:0
-ret: 0 st: 1 dts:0.026000 pts:0.026000 pos:28818 size:209 flags:1
+ret:-5
 ret: 0 st: 1 ts:1.471000 flags:1
-ret: 0 st: 0 dts:0.040000 pts:0.040000 pos:29034 size:10153 flags:0
+ret:-5
 ret: 0 st:-1 ts:0.365002 flags:0
 ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:478 size:28118 flags:1
 ret: 0 st:-1 ts:-0.740831 flags:1
-ret: 0 st: 1 dts:0.000000 pts:0.000000 pos:28603 size:208 flags:1
+ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:478 size:28118 flags:1
 ret: 0 st: 0 ts:2.153000 flags:0
-ret: 0 st: 1 dts:0.026000 pts:0.026000 pos:28818 size:209 flags:1
+ret:-5
 ret: 0 st: 0 ts:1.048000 flags:1
 ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:478 size:28118 flags:1
 ret: 0 st: 1 ts:-0.058000 flags:0
-ret: 0 st: 1 dts:0.000000 pts:0.000000 pos:28603 size:208 flags:1
+ret:-5
 ret: 0 st: 1 ts:2.836000 flags:1
-ret: 0 st: 1 dts:0.026000 pts:0.026000 pos:28818 size:209 flags:1
+ret:-5
 ret: 0 st:-1 ts:1.730004 flags:0
-ret: 0 st: 0 dts:0.040000 pts:0.040000 pos:29034 size:10153 flags:0
+ret:-5
 ret: 0 st:-1 ts:0.624171 flags:1
 ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:478 size:28118 flags:1
 ret: 0 st: 0 ts:-0.482000 flags:0
@@ -2988,9 +2988,9 @@
 ret: 0 st: 0 ts:2.413000 flags:1
 ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:478 size:28118 flags:1
 ret: 0 st: 1 ts:1.307000 flags:0
-ret: 0 st: 1 dts:0.000000 pts:0.000000 pos:28603 size:208 flags:1
+ret:-5
 ret: 0 st: 1 ts:0.201000 flags:1
-ret: 0 st: 1 dts:0.026000 pts:0.026000 pos:28818 size:209 flags:1
+ret:-5
 ret: 0 st:-1 ts:-0.904994 flags:0
 ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:478 size:28118 flags:1
 ret: 0 st:-1 ts:1.989173 flags:1
@@ -2998,15 +2998,15 @@
 ret: 0 st: 0 ts:0.883000 flags:0
 ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:478 size:28118 flags:1
 ret: 0 st: 0 ts:-0.222000 flags:1
-ret: 0 st: 1 dts:0.000000 pts:0.000000 pos:28603 size:208 flags:1
+ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:478 size:28118 flags:1
 ret: 0 st: 1 ts:2.672000 flags:0
-ret: 0 st: 1 dts:0.026000 pts:0.026000 pos:28818 size:209 flags:1
+ret:-5
 ret: 0 st: 1 ts:1.566000 flags:1
-ret: 0 st: 0 dts:0.040000 pts:0.040000 pos:29034 size:10153 flags:0
+ret:-5
 ret: 0 st:-1 ts:0.460008 flags:0
 ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:478 size:28118 flags:1
 ret: 0 st:-1 ts:-0.645825 flags:1
-ret: 0 st: 1 dts:0.000000 pts:0.000000 pos:28603 size:208 flags:1
+ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:478 size:28118 flags:1
 ----------------
 tests/data/b-libav.mmf
 ret: 0 st: 0 dts:0.000000 pts:0.000000 pos:-1 size:4096 flags:1

seek regression test: error


-- 
	-Mike Melanson