No duration when EXTINF without comma in the end about m3u8

[grafov]

For sure are you about comma in the end? So I have edited the title.

[bradleyfalzon]

I agree @grafov, it seems safe to be a little less strict. Something like:

diff --git a/reader.go b/reader.go
index cabe0a8..7f035ea 100644
--- a/reader.go
+++ b/reader.go
@@ -358,7 +358,10 @@ func decodeLineOfMediaPlaylist(p *MediaPlaylist, wv *WV, state *decodingState, l
                state.listType = MEDIA
                sepIndex := strings.Index(line, ",")
                if sepIndex == -1 {
-                       break
+                       if strict {
+                               return fmt.Errorf("could not parse: %q", line)
+                       }
+                       sepIndex = len(line)
                }
                duration := line[8:sepIndex]
                if len(duration) > 0 {

This will now error when in strict mode instead of silently discarding the EXTINF duration, and allows the user to switch to a non-strict parsing to accept the bad input.

Note, I did try to refactor the switch statement slightly, but every time there was a performance hit, and we have a couple users concerned about the speed, so I'm happy with maintaining the change as it is.

[grafov]

It looks like these playlists don't support standard. So then it is not a bug but a feature for the library - support broken playlists. I think support of these broken playlists will not affect valid playlists so we can add this "feature". What do you think @bradleyfalzon?

[igk1972]

@grafov
I was wrong, of course it's a comma

[bradleyfalzon]

OK, have a look at #85 and see if that's suitable.