Fix WebVTT milliseconds separator (should be . it seems, not ,) v1.6.39
authorCarl Hetherington <cth@carlh.net>
Sun, 13 Nov 2022 23:39:25 +0000 (00:39 +0100)
committerCarl Hetherington <cth@carlh.net>
Sun, 13 Nov 2022 23:39:25 +0000 (00:39 +0100)
src/web_vtt_reader.cc
test/data/test.vtt

index 2e772f143b99a8e65cdd0de325e500758faf9af3..cd6844de48d45ee6e63ea8fee55e53f451b91975 100644 (file)
@@ -111,13 +111,13 @@ WebVTTReader::read(std::function<optional<string> ()> get_line)
                                }
 
                                string expected;
-                               auto from = SubripReader::convert_time(parts[0], &expected);
+                               auto from = SubripReader::convert_time(parts[0], ".", &expected);
                                if (!from) {
                                        throw WebVTTError(parts[0], expected, _context);
                                }
                                rs.from = *from;
 
-                               auto to = SubripReader::convert_time(parts[2], &expected);
+                               auto to = SubripReader::convert_time(parts[2], ".", &expected);
                                if (!to) {
                                        throw WebVTTError(parts[2], expected, _context);
                                }
index 6cd60eb709fab6bdf30552a8c13e9d03ee49cd33..15475d4c921c80af150a586604d2572d34719942 100644 (file)
@@ -5,9 +5,9 @@ Seem: to put stuff here
 NOTE You can have notes
 That span multiple lines
 
-00:00:41,090 --> 00:00:42,210
+00:00:41.090 --> 00:00:42.210
 This is a subtitle
 and that's a line break
 
-00:01:01,010 --> 00:01:02,100
+00:01:01.010 --> 00:01:02.100
 This is some stuff.