RepoMirrors/mpv
1
0
Fork 0
mirror of https://github.com/mpv-player/mpv synced 2025-04-11 04:01:31 +00:00
Code Issues Releases Wiki Activity

vo_opengl: avoid outputting ultra-wide-gamut by default

Browse Source 
The default behavior of vo_opengl has pretty much always been 'show the
source colors as-is, without caring to adapt it to the target device'.
This decision is mostly based on the fact that if we do anything else,
lots of people will complain.

With the rise of content like BT.2020, however, it turns out more people
complain about this content being very desaturated than people complain
about this content not matching VLC - so let's just map ultra-wide gamut
content back down to standard gamut by default.
This commit is contained in:
Niklas Haas 2016-06-07 11:31:22 +02:00 committed by wm4
parent 9bd886f449
commit 38ac5d5e7b
1 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
video/out/opengl/video.c
View File

@ -2245,8 +2245,19 @@ static void pass_colormanage(struct gl_video *p, float peak_src,
}
}
if (prim_dst == MP_CSP_PRIM_AUTO)
// When auto-guessing the output color params, just pick the source color
// params to preserve the authentic "look and feel" of wrong/naive players.
// Some exceptions apply to source spaces that even hardcore technoluddites
// would probably not enjoy viewing unaltered
if (prim_dst == MP_CSP_PRIM_AUTO) {
prim_dst = prim_src;
// Avoid outputting very wide gamut content automatically, since the
// majority target audience has standard gamut displays
if (prim_dst == MP_CSP_PRIM_BT_2020 || prim_dst == MP_CSP_PRIM_PRO_PHOTO)
prim_dst = MP_CSP_PRIM_BT_709;
}
if (trc_dst == MP_CSP_TRC_AUTO) {
trc_dst = trc_src;
// Avoid outputting linear light at all costs. First try
@ -2260,6 +2271,7 @@ static void pass_colormanage(struct gl_video *p, float peak_src,
if (trc_dst == MP_CSP_TRC_LINEAR || trc_dst == MP_CSP_TRC_SMPTE_ST2084)
trc_dst = MP_CSP_TRC_GAMMA22;
}
if (!peak_src) {
// If the source has no information known, it's display-referred
// (and should be treated relative to the specified desired peak_dst)