diff options
| author | Kacper Michajłow <kasper93@gmail.com> | 2023-08-07 17:10:34 +0200 |
|---|---|---|
| committer | Niklas Haas <github-daiK1o@haasn.dev> | 2023-08-07 18:22:41 +0200 |
| commit | 1b10d9dd9cbc4164ed0ddbba0b64e64e638f1a1f (patch) | |
| tree | fdd9ad4cf94c6b8a2904ba3b6424c0e913b391f2 /video | |
| parent | 35a6b26b780af4befa4da06a63bebf95571d1279 (diff) | |
| download | mpv-1b10d9dd9cbc4164ed0ddbba0b64e64e638f1a1f.tar.bz2 mpv-1b10d9dd9cbc4164ed0ddbba0b64e64e638f1a1f.tar.xz | |
mp_image: properly infer color levels for some pixfmts
ffmpeg does not tag yuv levels for GRAY formats, but apparently they
should be infered as full range. Instead of defaulting to limited range
always. Fixes (M)JPEG playback.
This mimic ffmpeg's behaviour.
See: https://github.com/FFmpeg/FFmpeg/blob/d295b6b6936c9c691a4871851fdaec04d407e54b/libswscale/utils.c#L926-L962
Fixes: #12089
Diffstat (limited to 'video')
| -rw-r--r-- | video/mp_image.c | 30 |
1 files changed, 29 insertions, 1 deletions
diff --git a/video/mp_image.c b/video/mp_image.c index 40678d10bd..c55b205ec0 100644 --- a/video/mp_image.c +++ b/video/mp_image.c @@ -837,6 +837,34 @@ void mp_image_set_attributes(struct mp_image *image, mp_image_set_params(image, &nparams); } +static enum mp_csp_levels infer_levels(enum mp_imgfmt imgfmt) +{ + switch (imgfmt2pixfmt(imgfmt)) { + case AV_PIX_FMT_YUVJ420P: + case AV_PIX_FMT_YUVJ411P: + case AV_PIX_FMT_YUVJ422P: + case AV_PIX_FMT_YUVJ444P: + case AV_PIX_FMT_YUVJ440P: + case AV_PIX_FMT_GRAY8: + case AV_PIX_FMT_YA8: + case AV_PIX_FMT_GRAY9LE: + case AV_PIX_FMT_GRAY9BE: + case AV_PIX_FMT_GRAY10LE: + case AV_PIX_FMT_GRAY10BE: + case AV_PIX_FMT_GRAY12LE: + case AV_PIX_FMT_GRAY12BE: + case AV_PIX_FMT_GRAY14LE: + case AV_PIX_FMT_GRAY14BE: + case AV_PIX_FMT_GRAY16LE: + case AV_PIX_FMT_GRAY16BE: + case AV_PIX_FMT_YA16BE: + case AV_PIX_FMT_YA16LE: + return MP_CSP_LEVELS_PC; + default: + return MP_CSP_LEVELS_TV; + } +} + // If details like params->colorspace/colorlevels are missing, guess them from // the other settings. Also, even if they are set, make them consistent with // the colorspace as implied by the pixel format. @@ -861,7 +889,7 @@ void mp_image_params_guess_csp(struct mp_image_params *params) if (params->color.gamma == MP_CSP_TRC_V_LOG) { params->color.levels = MP_CSP_LEVELS_PC; } else { - params->color.levels = MP_CSP_LEVELS_TV; + params->color.levels = infer_levels(params->imgfmt); } } if (params->color.primaries == MP_CSP_PRIM_AUTO) { |