Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating tags in section 4.4.2 of draft 16 #148

Conversation

theRealRobG
Copy link
Collaborator

Description

This PR brings section 4.4.2 Media or Multivariant Playlist Tags up to date with draft 16 of the specification.

The main change here is the addition of the EXT-X-DEFINE tag and associated validators (both tag and playlist validators).

A further improvement may be to add handling for variable substitution into mamba, as defined in section 4.3:
https://datatracker.ietf.org/doc/html/draft-pantos-hls-rfc8216bis-16#section-4.3

The difficulty there would be in implementing the IMPORT definition, as it could not be self-contained to one parsed playlist, and would need a way of associating another parsed multivariant playlist, or at least the key-value pairs that were obtained from it.

Change Notes

  • Added new PantosTag to represent the EXT-X-DEFINE tag.
  • Added both tag and playlist validators based on the specification rules defined for this tag.

Pre-submission Checklist

  • I ran the unit tests locally before checking in.
  • I made sure there were no compiler warnings before checking in.
  • I have written useful documentation for all public code.
  • I have written unit tests for this new feature.

Media or Multivariant Playlist tags are defined in section 4.4.2:
https://datatracker.ietf.org/doc/html/draft-pantos-hls-rfc8216bis-16#section-4.4.2

This commit updates the existing tags in that section to be in line with
draft 16.

The main change here is the addition of the EXT-X-DEFINE tag and
associated validators (both tag and playlist validators).

A further improvement may be to add handling for variable substitution
into mamba, as defined in section 4.3:
https://datatracker.ietf.org/doc/html/draft-pantos-hls-rfc8216bis-16#section-4.3

The difficulty there would be in implementing the IMPORT definition, as
it could not be self-contained to one parsed playlist, and would need a
way of associating another parsed multivariant playlist, or at least the
key-value pairs that were obtained from it.
@theRealRobG theRealRobG requested a review from rmigneco December 1, 2024 03:18
@theRealRobG theRealRobG self-assigned this Dec 1, 2024
@theRealRobG theRealRobG merged commit c193dd9 into develop_1.x Dec 4, 2024
4 checks passed
@theRealRobG theRealRobG deleted the missing-multivariant-or-media-playlist-tags-from-draft-16 branch December 4, 2024 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants