pixel_checker: Move sRGB conversion into get_limits()

The sRGB conversion has to be done every time the limits are being
computed. Without this fix, pixel_checker_get_min/max() will produce
the wrong results when called from somewhere other than
pixel_checker_check().

diff --git a/test/utils.c b/test/utils.c
index 85b58d038d12fe3c154777a3a2f7f4ca31b22a4f..df16ef5fc470a4d08d5588ccd438c42a7eb36171 100644 (file)
--- a/test/utils.c
+++ b/test/utils.c
@@ -962,6 +962,18 @@ get_limits (const pixel_checker_t *checker, double limit,
            color_t *color,
            int *ao, int *ro, int *go, int *bo)
 {
+    color_t tmp;
+
+    if (PIXMAN_FORMAT_TYPE (checker->format) == PIXMAN_TYPE_ARGB_SRGB)
+    {
+       tmp.a = color->a;
+       tmp.r = convert_linear_to_srgb (color->r);
+       tmp.g = convert_linear_to_srgb (color->g);
+       tmp.b = convert_linear_to_srgb (color->b);
+
+       color = &tmp;
+    }
+    
     *ao = convert (color->a + limit, checker->aw, checker->am, checker->as, 1.0);
     *ro = convert (color->r + limit, checker->rw, checker->rm, checker->rs, 0.0);
     *go = convert (color->g + limit, checker->gw, checker->gm, checker->gs, 0.0);
@@ -988,25 +1000,11 @@ pixel_checker_get_min (const pixel_checker_t *checker, color_t *color,
 
 pixman_bool_t
 pixel_checker_check (const pixel_checker_t *checker, uint32_t pixel,
-                    color_t *color_in)
+                    color_t *color)
 {
     int32_t a_lo, a_hi, r_lo, r_hi, g_lo, g_hi, b_lo, b_hi;
     int32_t ai, ri, gi, bi;
     pixman_bool_t result;
-    color_t tmp, *color;
-
-    if (PIXMAN_FORMAT_TYPE (checker->format) == PIXMAN_TYPE_ARGB_SRGB)
-    {
-       tmp.a = color_in->a;
-       tmp.r = convert_linear_to_srgb (color_in->r);
-       tmp.g = convert_linear_to_srgb (color_in->g);
-       tmp.b = convert_linear_to_srgb (color_in->b);
-       color = &tmp;
-    }
-    else
-    {
-       color = color_in;
-    }
 
     pixel_checker_get_min (checker, color, &a_lo, &r_lo, &g_lo, &b_lo);
     pixel_checker_get_max (checker, color, &a_hi, &r_hi, &g_hi, &b_hi);