Eyevinn / hls-m3u8

92%

main: 92%

LAST BUILD ON BRANCH fix/encode-mutates-head

branch: fix/encode-mutates-head

CHANGE BRANCH

x

Reset

• fix/encode-mutates-head
• 41-keys-are-written-before-discontinuity
• 43-weird-interaction-when-ext-x-cue-out-follows-ext-x-daterange
• 46-ext-x-skip-implementation-seems-incorrect
• 50-instream-id-for-all-media-types
• accept-trailing-scte35-daterange
• allow-cache-obsolete
• byterange-offset
• chore/fix-tests
• chore/remove-goreportcard-badge
• ci/node24-actions
• content-steering
• define-tag
• docs-060
• docs-draft-v19
• docs-update
• docs/changelog-pr80
• docs/v0.6.1
• docs/v0.6.3
• docs/v0.6.4
• draft-22
• expanded-channels-support
• feat-calc-min-ver
• feat/support-low-latency
• feat/write-precision
• fix-65-get-all-segments
• fix/duration-attribute-on-cue-out
• fix/encode-ring-wraparound
• gap-support
• main
• multi-SCTE35-DateRange
• multiple-keys
• no-regex
• pooled-buffers
• refactor-and-fix
• release/v0.6.5
• server-control
• session-key
• sessiondata
• targetduration-int
• test-dangling-daterange
• update-changelog
• update-go-125
• update-tests
• v0.2.0
• v0.3.0
• v0.4
• v0.5.0-changelog
• v0.5.2-docs
• variant-params
• ver0-6-2
• verify-preload-interstitials

Build # 30232595495

Build Type

Pull #90

github

Committed by nvkhoi112358

Commit Message

fix: Encode() must not mutate playlist head pointer

Encode() was mutating p.head (introduced in 882d0c5, PR #33),
making it destructive — a second Encode() call on the same
MediaPlaylist produced an empty playlist because the head pointer
had been shifted past all segments.

The local variable `start` already computes the correct starting
index for the output loop. Remove the p.head mutation to restore
idempotent Encode() behavior.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Pull Request Pull Request #90: fix: Encode() must not mutate playlist head pointer

Coverage Stats

2242 of 2429 relevant lines covered (92.3%)

1.08 hits per line