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

Fix repeated segments #1489

Merged
merged 22 commits into from
Feb 28, 2024
4 changes: 2 additions & 2 deletions src/util/media-sequence-sync.js
Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,8 @@
class SyncInfoData {
/**
*
* @param {SyncInfo} segmentSyncInfo
* @param {Array<SyncInfo>} [partsSyncInfo]
* @param {SyncInfo} segmentSyncInfo - sync info for a give segment
dzianis-dashkevich marked this conversation as resolved.
Show resolved Hide resolved
* @param {Array<SyncInfo>} [partsSyncInfo] - sync infos for a list of parts for a given segment
*/
constructor(segmentSyncInfo, partsSyncInfo = []) {
this.segmentSyncInfo_ = segmentSyncInfo;
Expand All @@ -69,8 +69,8 @@
return this.partsSyncInfo_;
}

get hasPartsSyncInfo() {
return this.partsSyncInfo_.length > 0;

Check warning on line 73 in src/util/media-sequence-sync.js

View check run for this annotation

Codecov / codecov/patch

src/util/media-sequence-sync.js#L72-L73

Added lines #L72 - L73 were not covered by tests
}

resetAppendStatus() {
Expand All @@ -96,8 +96,8 @@
return this.start_;
}

get end() {
return this.end_;

Check warning on line 100 in src/util/media-sequence-sync.js

View check run for this annotation

Codecov / codecov/patch

src/util/media-sequence-sync.js#L99-L100

Added lines #L99 - L100 were not covered by tests
}

get diagnostics() {
Expand Down
Loading