Support relaxed parsing about node-webvtt HOT 3 CLOSED

@qgustavor this is fixed in #24 and released in version 1.6.1

from node-webvtt.

Hi @qgustavor, thanks for this.

Originally this project was made to create HLS playlists from webvtt files and as such, isn't a fully fledged parser. However, I'm ready to do the changes needed to support whatever makes sense for its' users!

I'll look into this, you are also more than welcome to open a PR where we can collaborate if you're in a hurry ;)

from node-webvtt.

I made some changes here: qgustavor@ed62d47

Those add an errors property to the parser result with the errors thrown curing cue parsing process and change the value of valid to false if some error was thrown. Cues that could be parsed are also returned.

I'm not sure if the way I made those changes are OK for you, so I'm not sending a pull request by now. Maybe the parseCues function got too messed.

Edit: About the files with cue parsing issues I found that those're converted from .ass files and the conversion code don't handled blank lines well, so I need to fix it before handling the .vtt files. Anyway, even if it don't need it anymore I think supporting relaxed parsing would a good improvement.

from node-webvtt.