Repository navigation
feat(video): add selectable video codecs - #194
ReenigneArcher wants to merge 1 commit into
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #194 +/- ##
==========================================
- Coverage 63.16% 62.83% -0.33%
==========================================
Files 35 33 -2
Lines 5261 5330 +69
Branches 2385 2417 +32
==========================================
+ Hits 3323 3349 +26
- Misses 935 964 +29
- Partials 1003 1017 +14
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 6 files with indirect coverage changes Continue to review full report in Codecov by Harness.
|
0927719 to
5320639
Compare
c00fc4c to
5a2991d
Compare
e8326cd to
dff1ed2
Compare
dff1ed2 to
7e1f16d
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #194 +/- ##
==========================================
- Coverage 65.10% 65.02% -0.09%
==========================================
Files 49 50 +1
Lines 5875 5975 +100
Branches 2621 2654 +33
==========================================
+ Hits 3825 3885 +60
- Misses 969 1188 +219
+ Partials 1081 902 -179
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 11 files with indirect coverage changes Continue to review full report in Codecov by Harness.
|
7e1f16d to
738fc4b
Compare
bd1d409 to
0f96482
Compare
0f96482 to
1364172
Compare
1364172 to
e311e42
Compare
d7636d9 to
90041bb
Compare
90041bb to
934387a
Compare
|



Description
Add H.264, MPEG-2/H.262, and H.263+ video codec selection. Auto prefers H.264, then MPEG-2, then H.263+; an explicit selection requests only that codec and fails negotiation when the host cannot provide it.
streaming.video_codec, and continue reading existingstreaming.video_decoderpreferences.#339 and #340 are merged. This PR is rebased directly onto current
master(ff81ce0) and contains only the codec-selection commit. It uses the controller handling, GPU/software presentation, and live statistics already on master.The common-library pin remains
ed6b2da, based on upstream94c65fd. This PR still depends on moonlight-common-c #175, which is open. Related host support is tracked in Sunshine #5194.TODO:
Screenshot
Issues Fixed or Closed
Roadmap Issues
Testing
ctest --test-dir cmake-build-release --output-on-failure: 294/294 passed, including settings migration, codec negotiation, wire-format, controller, and presenter tests.git range-diffconfirms the single codec-selection commit is unchanged by the rebase.git diff --exit-code 90041bb HEADis empty: the complete resulting tree, including dependency pins, is identical to the previous PR head.apidocumentation build (441 HTML pages), formatting, and all 17 CMake lint checks.90041bb, including Linux/macOS/Windows CI, Common Lint, CodeQL, SonarCloud, and Codecov. Checks on the new head will rerun after the push.ed6b2da; master's newer toml++ remainscfd7159.Real Xbox hardware and interactive emulator validation were not repeated for this history-only rebase.
Type of Change
Checklist
AI Usage
See our AI usage policy.
AI tools generated much of the feature and assisted with implementation review, the shared protocol changes, settings migration, rebasing, documentation, and tests.