commit b97a9194186a30e6e56ae4173b197d3e062fd661
author Mike Klein <mtklein@google.com> Fri Nov 08 14:00:00 2019 -0600
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Fri Nov 08 20:40:47 2019 +0000
tree a530face620613fd678c6ba571dc9fdfd8fa4943
parent a53e47fe9483117e65e05a006e88c2cccf974a1a

don't clamp alpha, it's already [0,1]

No diffs.

Change-Id: I13a848955605a16aa229b98a14dc0e1930ba28c2
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/253721
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>

src/core/SkVMBlitter.cpp

diff --git a/src/core/SkVMBlitter.cpp b/src/core/SkVMBlitter.cpp
index a637049..9c2a5702 100644
--- a/src/core/SkVMBlitter.cpp
+++ b/src/core/SkVMBlitter.cpp
@@ -306,8 +306,7 @@
                 src.r = min(max(splat(0.0f), src.r), splat(1.0f));
                 src.g = min(max(splat(0.0f), src.g), splat(1.0f));
                 src.b = min(max(splat(0.0f), src.b), splat(1.0f));
-                src.a = min(max(splat(0.0f), src.a), splat(1.0f));
-                // TODO: remove src.a clamp?  alpha shouldn't go outside [0,1].
+                // src.a should already be in [0,1].
             }
             if (force_opaque) { src.a = splat(1.0f); }