You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Dec 8, 2024. It is now read-only.
From a few cursory tests, it seems it doesn't handle variants correctly in 2 ways..
it only adds the Alternatives to the first media playlist it fills in as a variant.
If the alternatives are at the end of the m3u8, none of them get it.
It would seem the fix would be to only accuulate "alternatives" until the file has been fully parsed, and then to use the accuulated alternatives to each Variant.
Another possible fix would be to simply not include the Alternatives in the Variant, but have them at the same level in the struct as the Variants array.
This assume that "EXT-X-MEDIA" can be really anywhere in the file, its possible that they should always be before the the media playlist reference, but in practce I have seen files that have them at the top and at the bottom. The first issue I saw with the file at the top, while the second issue with them at the bottom)).
The text was updated successfully, but these errors were encountered:
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
From a few cursory tests, it seems it doesn't handle variants correctly in 2 ways..
It would seem the fix would be to only accuulate "alternatives" until the file has been fully parsed, and then to use the accuulated alternatives to each Variant.
Another possible fix would be to simply not include the Alternatives in the Variant, but have them at the same level in the struct as the Variants array.
This assume that "EXT-X-MEDIA" can be really anywhere in the file, its possible that they should always be before the the media playlist reference, but in practce I have seen files that have them at the top and at the bottom. The first issue I saw with the file at the top, while the second issue with them at the bottom)).
The text was updated successfully, but these errors were encountered: