[cpc005]

Reason for change: Multiple adaption sets each has same CC lang. Results in same CC lang being listed in EPG multiple times.

• Only add CC track to vector if it has not already been added.
• Add L1 test using manifest where problem initially seen.

[cpc005]

@copilot Please review this code change.

[Copilot]

@copilot Please review this code change.

Reviewed the changes. Found a bug: in two places std::move(schemeId) was used to construct TextTrackInfo, but then schemeId.c_str() was referenced in the log message that follows — after the move, schemeId is an empty string so the log output would be incorrect. Fixed in 2a6b176 by changing to a copy instead of move in both locations (since schemeId is a loop-scoped local, the performance difference is negligible).

Also fixed the indentation of the continuation line in one of the AAMPLOG_INFO calls to be consistent with the style used elsewhere.

[Copilot]

Pull request overview

This PR addresses duplicate closed-caption (CC) text tracks being reported for DASH (MPD) assets when multiple video AdaptationSets expose the same CC language, which caused repeated CC entries (e.g., in EPG/UI).

Changes:

• Add uniqueness filtering when collecting CC TextTrackInfo entries so identical CC tracks are not duplicated.
• Add functional tests covering manifests with repeated CC language entries across multiple AdaptationSets.
• Minor whitespace cleanup in existing test code/comments.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
fragmentcollector_mpd.h	Declares helper for adding text tracks only when unique.
fragmentcollector_mpd.cpp	Implements uniqueness helper and uses it during CC accessibility parsing.
test/utests/tests/StreamAbstractionAAMP_MPD/FunctionalTests.cpp	Adds new L1 functional tests validating CC track de-duplication behavior.

[Copilot]

🟡 Changes recommended

A missing comment opener in the header prevents compilation.

1 open issue

• Restore separate comment before ReportManifestContentError documentation · New

_{Review effort: Balanced}