11646 2011-01-02 21:02:24 +0000 step 13: copypasta in "the character at position is not a U+003A COLON character" 2011-08-04 05:14:17 +0000 1 1 1 Unclassified HTML WG LC1 HTML5 spec unspecified Other other RESOLVED FIXED http://www.whatwg.org/specs/web-apps/current-work/#parsing-0 P3 normal --- 1 contributor ian ian mike philipj public-html-admin public-html-wg-issue-tracking public-html-bugzilla oldest_to_newest 43760 0 contributor 2011-01-02 21:02:24 +0000 Specification: http://www.whatwg.org/specs/web-apps/current-work/multipage/video.html Section: http://www.whatwg.org/specs/web-apps/current-work/#parsing-0 Comment: step 13: copypasta in "the character at position is not a U+003A COLON character" Posted from: 81.234.240.182 43762 1 philipj 2011-01-02 21:21:53 +0000 From context, it seems that the "not" shouldn't be there. It seems to me that the parsing of WebVTT timestamps is more convoluted than it needs to be, and that the reason is that hours can be any number of digits. In SRT it is exactly 2 and I think just going with that would be simpler. Allowing an arbitrary number of digits just makes parsing more complex and forces implementors to handle overflow in timestamps like 99999999999999999999999999:00:00.000 and using greater precision than really necessary. There's also extra logic thrown in to not allow single-digit minutes in step 13.1. I'd prefer exactly 2 digits for hours, limiting WebVTT to use with video files of duration shorter than 100 hours, which is hardly a practical problem. 43765 2 philipj 2011-01-02 21:24:25 +0000 Opened a separate bug for the issue of 2-digit hours: http://www.w3.org/Bugs/Public/show_bug.cgi?id=11647 45099 3 ian 2011-02-03 21:11:53 +0000 EDITOR'S RESPONSE: This is an Editor's Response to your comment. If you are satisfied with this response, please change the state of this bug to CLOSED. If you have additional information and would like the editor to reconsider, please reopen this bug. If you would like to escalate the issue to the full HTML Working Group, please add the TrackerRequest keyword to this bug, and suggest title and text for the tracker issue; or you may create a tracker issue yourself, if you are able to do so. For more details, see this document: http://dev.w3.org/html5/decision-policy/decision-policy.html Status: Accepted Change Description: see diff given below Rationale: Concurred with reporter's comments. 45103 4 ian 2011-02-03 21:14:54 +0000 Checked in as WHATWG revision r5825. Check-in comment: VTT: copypasta that made the parsing of timestamps fail if they had an hour component. http://html5.org/tools/web-apps-tracker?from=5824&to=5825 53483 5 mike 2011-08-04 05:14:17 +0000 mass-move component to LC1