diff options
| author | Jon Lund Steffensen <jonlst@gmail.com> | 2013-10-21 15:26:47 -0400 |
|---|---|---|
| committer | Jon Lund Steffensen <jonlst@gmail.com> | 2013-10-21 15:29:18 -0400 |
| commit | 95d8a3f7600073b4abe869078cf3db1d8497c60a (patch) | |
| tree | 12ee842fcf733f8eb3978280e24c44226d470183 /src/colorramp.c | |
| parent | Fix slight wobble in temperature caused by float. (diff) | |
| download | redshift-ng-95d8a3f7600073b4abe869078cf3db1d8497c60a.tar.gz redshift-ng-95d8a3f7600073b4abe869078cf3db1d8497c60a.tar.bz2 redshift-ng-95d8a3f7600073b4abe869078cf3db1d8497c60a.tar.xz | |
colorramp: Fix slight flicker caused by improper scaling of gamma ramp values
Values generated by colorramp_fill() will be a maximum of i * 256 - 1, due to the use of UINT16_MAX. Simplest fix ought to be using (UINT16_MAX+1), which is safe, as the maximum value (255 * 256) still fits in a uint16.
(Thanks to Sean Hildebrand)
Diffstat (limited to 'src/colorramp.c')
| -rw-r--r-- | src/colorramp.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/colorramp.c b/src/colorramp.c index ed399f8..4d32d7e 100644 --- a/src/colorramp.c +++ b/src/colorramp.c @@ -139,10 +139,10 @@ colorramp_fill(uint16_t *gamma_r, uint16_t *gamma_g, uint16_t *gamma_b, for (int i = 0; i < size; i++) { gamma_r[i] = pow((float)i/size, 1.0/gamma[0]) * - UINT16_MAX * brightness * white_point[0]; + (UINT16_MAX+1) * brightness * white_point[0]; gamma_g[i] = pow((float)i/size, 1.0/gamma[1]) * - UINT16_MAX * brightness * white_point[1]; + (UINT16_MAX+1) * brightness * white_point[1]; gamma_b[i] = pow((float)i/size, 1.0/gamma[2]) * - UINT16_MAX * brightness * white_point[2]; + (UINT16_MAX+1) * brightness * white_point[2]; } } |