diff options
| author | Oneric <oneric@oneric.stub> | 2022-11-02 16:45:46 +0100 |
|---|---|---|
| committer | Oneric <oneric@oneric.stub> | 2022-11-02 17:44:48 +0100 |
| commit | 113649bc9717909adcc44b246ac8e40c90391b10 (patch) | |
| tree | f0ede18da32d2af6c149e560183fb6e43cf5d60a | |
| parent | 4eb5c6dcff48628772568a7a185fbb1b0e57c014 (diff) | |
| download | libass-113649bc9717909adcc44b246ac8e40c90391b10.tar.bz2 libass-113649bc9717909adcc44b246ac8e40c90391b10.tar.xz | |
doc: fix description of MPC-HC ISR's colour mangling
I misunderstood what MPC-HC's ColorConvTable::NONE (not used for
"YCbCr Matrix: None"!) does and I'm not sure why I thought empty
headers would rely on resolution-based guessing.
After bringing up support for "None" in MPC-HC ISR with clsid2,
it was explained to me that ::NONE is currently equal to BT.601.
| -rw-r--r-- | libass/ass_types.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/libass/ass_types.h b/libass/ass_types.h index f235990..15d6d8a 100644 --- a/libass/ass_types.h +++ b/libass/ass_types.h @@ -197,9 +197,9 @@ typedef struct ass_event { * Keep in mind though, that xy-VSFilter cannot accurately implement this and * will instead resort to a guessing the video colorspace based on resolution * and then convert RGB to the guessed space. - * Also some versions of MPC-HC's Internal Subtitle Renderer don't (explicitly) - * implement "None", but use xy-VSFilter-like resolution-based guessing for - * unknown values or no header at all (which ofc also breaks old subtitles). + * Also some versions of MPC-HC's Internal Subtitle Renderer don't implement + * "None" and use TV.601 for unknown, but the video colourspace for no or an + * empty header (which can break old subtitles). * * Aegisub's (the main application to produce ASS subtitle scripts) behaviour * regarding colorspaces is unfortunately a bit confusing. |