docs: correct DV routing, CLI flags, and source map against 4.6.3#84
Merged
Conversation
Audited README + docs/{architecture,formats,cli}.md against the actual
sources at HEAD (4.6.3) and fixed the drift found:
formats.md (Dolby Vision signaling was the biggest miss):
- DV routing table no longer collapses P5/P8.1/P8.4 under "dvh1/dvhe".
P5 emits a bare dvh1 primary; P8.1/P8.4/P7 emit hvc1 primary + dvvC
with dvh1.08/db1p or /db4h in SUPPLEMENTAL-CODECS on DV panels, plain
HDR10/HLG base elsewhere. dvhe is never emitted.
- AV1+DV corrected: P10.0/P10.1 are bare dav1 with no supplemental,
only P10.4 carries the dav1/db4h supplemental.
- Documented the SDR-compatible-base profiles (HEVC P8.2, AV1 P10.2)
that strip the DV config and play the Rec.709 base on every panel.
cli.md:
- "Fifteen subcommands" was off by one (sixteen).
- hlslive --segments is a required comma-separated list of .ts paths,
not a numeric sizing flag; clarified --seconds/--segment-seconds/--disc.
- serve --native-subs: the index value is legacy/ignored; the flag drives
requestNativeSubtitleTrack + attachAllNativeSubtitleStores, not
LoadOptions.prepareNativeSubtitles.
- Added the undocumented flags: live --serve-only/--rewind-test/
--gen-highbitrate-seed and customio --audio-only.
architecture.md source map:
- Added the omitted files: DiscRecognitionCache, HTTPDiscIOReader,
MovTextSampleBuilder, NativeSubtitleCueStore, DoviRpuConverter+Probe,
Issue65LivelockBreakers.
- Added the AetherEngineSMB target subtree and a pointer to cli.md for
the aetherctl target.
README.md:
- Audio bridge row now lists DTS-HD MA.
- Documented LoadOptions.nativeRemoteHLS for AVPlayer-native remote HLS.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XZTEfmztPE8hAdjHdBr9BH
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Audited the README and
docs/{architecture,formats,cli}.mdagainst the actual sources at HEAD (4.6.3) and fixed the drift found. Docs-only, no code changes.formats.md (the biggest miss: Dolby Vision signaling)
dvh1/dvhe". Reality perCodecRoutePolicy.swift: P5 emits a baredvh1primary; P8.1 / P8.4 / P7 emithvc1primary +dvvCwithdvh1.08/db1por/db4hinSUPPLEMENTAL-CODECSon DV panels, plain HDR10 / HLG base elsewhere.dvheis never emitted.dav1with no supplemental; only P10.4 carries thedav1/db4hsupplemental.cli.md
hlslive --segmentsis a required comma-separated list of.tspaths, not a numeric sizing flag; clarified--seconds/--segment-seconds/--disc.serve --native-subs: the<index>value is legacy and ignored; the flag drivesrequestNativeSubtitleTrack()+attachAllNativeSubtitleStores(), notLoadOptions.prepareNativeSubtitles.live --serve-only/--rewind-test/--gen-highbitrate-seed, andcustomio --audio-only.architecture.md source map
DiscRecognitionCache,HTTPDiscIOReader,MovTextSampleBuilder,NativeSubtitleCueStore,DoviRpuConverter+Probe,Issue65LivelockBreakers.AetherEngineSMBtarget subtree and a pointer to cli.md for theaetherctltarget.README.md
LoadOptions.nativeRemoteHLSfor AVPlayer-native remote HLS.Each claim was verified against the cited source (
CodecRoutePolicy.swift,Serve.swift,HLSLiveRepro.swift,main.swift, theSources/tree). All cross-doc anchors and file links resolve; no broken references.🤖 Generated with Claude Code