diff options
author | Uoti Urpala <uau@mplayer2.org> | 2011-08-29 05:38:44 +0300 |
---|---|---|
committer | Uoti Urpala <uau@mplayer2.org> | 2011-08-29 06:34:36 +0300 |
commit | b6628f4e1ece6e0b8dbd81cbfa5d6da30c73e824 (patch) | |
tree | 41474c4b26922f9e756d66da82fb3add8430c3a9 /Makefile | |
parent | 1478f658f332df960ac8960cd16b04000ad3cb89 (diff) | |
download | mpv-b6628f4e1ece6e0b8dbd81cbfa5d6da30c73e824.tar.bz2 mpv-b6628f4e1ece6e0b8dbd81cbfa5d6da30c73e824.tar.xz |
csputils/vo_gl: rewrite YUV->RGB matrix generation
Rewrite the csputils.c code generating a conversion matrix for
YUV->RGB conversions (currently used by vo_gl only). Functional
differences:
- The separate "mplayer default" colorspace is removed, and BT.601 is
used instead (the default colorspace was in fact BT.601; see below).
- The old code was missing chroma scaling. As a result the "mplayer
default" colorspace actually mapped to BT.601, and everything else
was buggy (I guess the other colorspaces were added with particular
coefficient semantics, without understanding that the original
"default colorspace" was actually BT.601 and why its coefficients
differed from the added version).
- The old code had a bug in the equalizer hue equations.
- The old code assumed that for specifying whether input and output
were limited-range or full-range YUV or RGB it would make sense to
specify "no conversion" meaning full-range YUV to full-range RGB or
limited-range YUV to limited-range RGB. This isn't true; limited-
range YUV has different ranges for luma and chroma (16-235
vs 16-240) which means you have to scale chroma for limited->limited
conversions. The new code assumes limited->limited conversions for
the levelconv parameter 2. It'd probably make sense to change the
API later to specify the ranges of input and output separately.
- The undocumented EBU and XYZ colorspaces are removed. I doubt any
videos use these. Also the EBU colorspace looks like it'd expect
a different input range - at least no input would map to full RGB
red as it was.
Diffstat (limited to 'Makefile')
0 files changed, 0 insertions, 0 deletions