[vlc-commits] Input clock reference: bump maximum gap to 60s

Rafaël Carré git at videolan.org
Mon Sep 2 17:48:49 CEST 2013


vlc/vlc-2.1 | branch: master | Rafaël Carré <funman at videolan.org> | Sat Aug 31 14:49:44 2013 +0200| [767dd93abf6e4f33ac40f43c5d4bd3ee0ffcf0fa] | committer: Jean-Baptiste Kempf

Input clock reference: bump maximum gap to 60s

Some APE files can have a single packet duration of up to 30s
Is there even a hard limit in this format, or can packets be longer?

Previous gap limit was an arbitrary 22s, written in a way reminiscent of
the MPEG 90kHz clock, which does not make sense anymore since reference clock
in all our demuxers use our own 1MHz clock.

Also explicitely use CLOCK_FREQ

(cherry picked from commit df958179f875265c2f3efc2a2484a015d93cfa8a)
Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>

> http://git.videolan.org/gitweb.cgi/vlc/vlc-2.1.git/?a=commit;h=767dd93abf6e4f33ac40f43c5d4bd3ee0ffcf0fa
---

 src/input/clock.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/input/clock.c b/src/input/clock.c
index 7d8b1e1..1467345 100644
--- a/src/input/clock.c
+++ b/src/input/clock.c
@@ -80,7 +80,7 @@
  *****************************************************************************/
 
 /* Maximum gap allowed between two CRs. */
-#define CR_MAX_GAP (INT64_C(2000000)*100/9)
+#define CR_MAX_GAP (60 * CLOCK_FREQ)
 
 /* Latency introduced on DVDs with CR == 0 on chapter change - this is from
  * my dice --Meuuh */



More information about the vlc-commits mailing list