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

Introduce EXT-X-SKIP support (Targeting version 2.x) #131

Merged
merged 6 commits into from
Sep 9, 2024

Conversation

theRealRobG
Copy link
Collaborator

Description

This PR cherry-picks the changes from #130 into a branch based out of develop to create a version compatible with version 2 of the library.

Change Notes

  • Added new value (EXT_X_SKIP) to the PantosTag enum and made the necessary updates for the HLSTagDescriptor conformance.
  • Added new values to PantosValue to account for attributes related to EXT-X-SKIP (.skippedSegments and .recentlyRemovedDateranges).
  • Updated generateMediaGroups for HLSPlaylistStructure to account for any SKIPPED-SEGMENTS when calculating media sequence values for segment tag groups.

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.

 Conflicts:
	mambaTests/PlaylistStructureAndEditingTests.swift
 Conflicts:
	mambaTests/PantosTagTests.swift
 Conflicts:
	mambaSharedFramework/HLS Models/Playlist Structure/HLSPlaylistStructure.swift
@theRealRobG theRealRobG added enhancement mamba2.0 A marker for features/bug fixes for a new version of mamba labels Aug 31, 2024
@theRealRobG theRealRobG self-assigned this Aug 31, 2024
 Conflicts:
	mambaSharedFramework/Playlist Models/Playlist Structure/PlaylistStructureCore.swift
@theRealRobG theRealRobG merged commit f56c79c into develop Sep 9, 2024
4 checks passed
@theRealRobG theRealRobG deleted the playlist-delta-updates-2.x branch September 9, 2024 21:48
@theRealRobG theRealRobG mentioned this pull request Oct 3, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement mamba2.0 A marker for features/bug fixes for a new version of mamba
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants