| commit | 6655095842ecdc56e1ab5507646dedd241e13d8b | [log] [tgz] |
|---|---|---|
| author | Mike Klein <mtklein@google.com> | Tue Aug 20 12:54:22 2019 -0500 |
| committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | Tue Aug 20 19:42:58 2019 +0000 |
| tree | 732b857427683984e7f85ad16ffae08df3d1a0ee | |
| parent | d931999c3afd98880d3f366e12f858824be44fba [diff] |
Add missing b-tweak to skcms_TF_Invert() First I added an assert (x >= 0) to powf_(). That failing helped me notice we're not checking ad+b >=0 in tf_is_valid(). That failing helped me notice skcms_TF_Invert() can take ad+b >= 0 inputs and create ad + b < 0 outputs. To fix that, apply the same tweak to b we do when fitting. To follow the story in smaller steps, 1) https://skia-review.googlesource.com/c/skcms/+/235899 2) https://skia-review.googlesource.com/c/skcms/+/235909 3) https://skia-review.googlesource.com/c/skcms/+/235912 Bug: chromium:976551 Change-Id: If281a2151f41b6dd5bcc86992d2104576f7c985f Reviewed-on: https://skia-review.googlesource.com/c/skcms/+/235913 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Mike Klein <mtklein@google.com>