diff options
| author | Mike Klein <mtklein@chromium.org> | 2017-06-09 09:32:24 -0400 |
|---|---|---|
| committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-06-09 14:18:12 +0000 |
| commit | 39afa1226f71d09bbdb21cc3c86cb23539d8311b (patch) | |
| tree | 1843db3d4e18ddea8256f85f39713db3aedd0b9b /samplecode/SampleShadowColor.cpp | |
| parent | eb86b7094755a5fc1e49d22b4c5323f372344d61 (diff) | |
fix f16 -> sRGB in encode_bitmap_for_png()
The existing logic looks wrong to me:
- clamp premul to [0,1]
- unpremul, ignoring zero alpha
It seems like we should do:
- unpremul, avoiding any divide by zero
- clamp unpremul to [0,1]
Am I misunderstanding this or has this just always been wrong?
Change-Id: I9636b9566c746bc05371e1e660f4e59dde16827b
Reviewed-on: https://skia-review.googlesource.com/19264
Reviewed-by: Matt Sarett <msarett@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'samplecode/SampleShadowColor.cpp')
0 files changed, 0 insertions, 0 deletions