commit 24c91d143c4a7d4191fd4ada7b4f08d48b5a1916
author Mike Klein <mtklein@chromium.org> Thu Apr 05 16:16:48 2018 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Apr 05 20:24:22 2018 +0000
tree 745c0740969e8a254c903787b31c879858879bcb
parent 6959d45972e2034b8c22d86a22dd232f549cd578

clean up some tests.c TODOs

Change-Id: Icccca933e7f86fec15173749a776f8418b092cde
Reviewed-on: https://skia-review.googlesource.com/118987
Commit-Queue: Mike Klein <mtklein@chromium.org>
Commit-Queue: Brian Osman <brianosman@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>

tests.c

diff --git a/tests.c b/tests.c
index 6cf4e6b..943064d 100644
--- a/tests.c
+++ b/tests.c
@@ -988,7 +988,7 @@
         dst, skcms_PixelFormat_RGBA_8888, skcms_AlphaFormat_PremulLinear, &sRGB,
         64));
     for (int i = 0; i < 256; i+=4) {
-        for (int k = 0; k < 1; k++) { // TODO: k < 3, why is only the red channel correct?
+        for (int k = 0; k < 3; k++) {
             float l = skcms_TransferFunction_eval(tf,       src[i+k]/255.0f);
             float e = skcms_TransferFunction_eval(&inv, l * src[i+3]/255.0f);
             expect( dst[i+k] == (uint8_t)(255.0f*e + 0.5f) );
@@ -1001,7 +1001,7 @@
         dst, skcms_PixelFormat_RGBA_8888, skcms_AlphaFormat_Unpremul    , &sRGB,
         64));
     for (int i = 0; i < 256; i+=4) {
-        for (int k = 0; k < 1; k++) { // TODO: k < 3, same question as above...
+        for (int k = 0; k < 3; k++) {
             float pm = skcms_TransferFunction_eval(tf,         src[i+k]/255.0f );
             float e  = skcms_TransferFunction_eval(&inv, pm / (src[i+3]/255.0f));
             expect( dst[i+k] == (uint8_t)(255.0f*e + 0.5f) );