diff options
| author | Jon Lund Steffensen <jonlst@gmail.com> | 2011-03-30 22:09:25 +0200 |
|---|---|---|
| committer | Jon Lund Steffensen <jonlst@gmail.com> | 2011-03-30 22:09:25 +0200 |
| commit | 54d7f9ff1c9c2e4a0b6a8957269285d6b9c8b37d (patch) | |
| tree | 40e00ca96057a28c3b9d7f6dbc7b962da2ae0c61 /src/gamma-w32gdi.c | |
| parent | config: Look in %userprofile%/.config/redshift.conf on windows platform. (diff) | |
| download | redshift-ng-54d7f9ff1c9c2e4a0b6a8957269285d6b9c8b37d.tar.gz redshift-ng-54d7f9ff1c9c2e4a0b6a8957269285d6b9c8b37d.tar.bz2 redshift-ng-54d7f9ff1c9c2e4a0b6a8957269285d6b9c8b37d.tar.xz | |
w32gdi: Remember to release the DC handle on error.
Add comment on failure of SetDeviceGammaRamp().
Diffstat (limited to 'src/gamma-w32gdi.c')
| -rw-r--r-- | src/gamma-w32gdi.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/src/gamma-w32gdi.c b/src/gamma-w32gdi.c index c5f760e..7ea8f42 100644 --- a/src/gamma-w32gdi.c +++ b/src/gamma-w32gdi.c @@ -144,6 +144,7 @@ w32gdi_set_temperature(w32gdi_state_t *state, int temp, float gamma[3]) WORD *gamma_ramps = malloc(3*GAMMA_RAMP_SIZE*sizeof(WORD)); if (gamma_ramps == NULL) { perror("malloc"); + ReleaseDC(NULL, hDC); return -1; } @@ -157,8 +158,12 @@ w32gdi_set_temperature(w32gdi_state_t *state, int temp, float gamma[3]) /* Set new gamma ramps */ r = SetDeviceGammaRamp(hDC, gamma_ramps); if (!r) { - fputs(_("Unable to set gamma ramps.\n"), stderr); + /* TODO it happens that SetDeviceGammaRamp returns FALSE on + occasions where the adjustment seems to be successful. + Does this only happen with multiple monitors connected? */ + fputs(_("Unable to set gamma ramps.\n"), stderr);s free(gamma_ramps); + ReleaseDC(NULL, hDC); return -1; } |