Merge pull request #118 from slifty/117-timestampfix

Support longer times
osk · May 12, 2022 · b9f1702 · b9f1702
2 parents 71ad68e + 4c60714
commit b9f1702
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/lib/parser.js b/lib/parser.js
@@ -11,7 +11,7 @@ function ParserError (message, error) {
 
 ParserError.prototype = Object.create(Error.prototype);
 
-const TIMESTAMP_REGEXP = /([0-9]{1,2})?:?([0-9]{2}):([0-9]{2}\.[0-9]{2,3})/;
+const TIMESTAMP_REGEXP = /([0-9]+)?:?([0-9]{2}):([0-9]{2}\.[0-9]{2,3})/;
 
 function parse (input, options) {
   if (!options) {

diff --git a/test/parser.test.js b/test/parser.test.js
@@ -105,6 +105,16 @@ a`;
     parse(input).cues[0].end.should.equal(0.001);
   });
 
+  it('should parse cue with long hours timestamp', () => {
+    const input = `WEBVTT
+
+10000:00:00.000 --> 10000:00:00.001
+a`;
+
+    parse(input).cues[0].start.should.equal(36000000);
+    parse(input).cues[0].end.should.equal(36000000.001);
+  });
+
   it('should return parsed data about a single cue', () => {
     const input = `WEBVTT