d2d1/tests: Use compare_uint() in compare_float() instead of abs().

The result of abs(INT_MIN) is INT_MIN, which breaks the ulps comparison.

d2d1/tests: Use compare_uint() in compare_float() instead of abs().
7febd8db · Jeff Smith · Alexandre Julliard · 0a8610f9 · 7febd8db
Commit 7febd8db authored Aug 02, 2023 by Jeff Smith Committed by Alexandre Julliard Aug 08, 2023
Show whitespace changes
Inline Side-by-side

Showing with 1 addition and 4 deletions

d2d1.c dlls/d2d1/tests/d2d1.c +1 -4

No files found.
--- a/dlls/d2d1/tests/d2d1.c
+++ b/dlls/d2d1/tests/d2d1.c
@@ -731,10 +731,7 @@ static BOOL compare_float(float f, float g, unsigned int ulps)
    if (y < 0)
        y = INT_MIN - y;
-    if (abs(x - y) > ulps)
+    return compare_uint(x, y, ulps);
-        return FALSE;
-    return TRUE;
 }
 static BOOL compare_colour_f(const D2D1_COLOR_F *colour, float r, float g, float b, float a, unsigned int ulps)