diff options
| author | Marek Olšák <[email protected]> | 2011-04-20 12:40:40 +0200 |
|---|---|---|
| committer | Marek Olšák <[email protected]> | 2011-04-22 06:35:36 +0200 |
| commit | 1faf079a692bbf4b24c8e83fa2b331c1e3b58e13 (patch) | |
| tree | 71192245f8e19e6517cfdc353b0af6bd832c69a5 /src/egl/drivers | |
| parent | 7142a67473b4fc77bdd58baf7cced9bfc54ccb73 (diff) | |
swrast: fix readpix clamping
Broken with e5c6a92a12b5cd7db205d72039f58d302b0be9d5. (ARB_color_buffer_float)
Clamping should occur if type != float, otherwise the MSBs of the resulting
pixels are killed off. For example, reading back LUMINANCE = R+G+B can be
greater than 0xff, but the result is naturally masked by 0xff
for UNSIGNED_BYTE, leading to bogus results.
The following bug report seems to want clamping to occur if type == half_float
too. Not sure what's correct.
Bug: [bisected pineview] oglc case pxconv-read failed
https://bugs.freedesktop.org/show_bug.cgi?id=35852
Tested by: Fang Xun <[email protected]>
Reviewed-and-tested-by: Ian Romanick <[email protected]>
Diffstat (limited to 'src/egl/drivers')
0 files changed, 0 insertions, 0 deletions