handle overflows in float->int
rects are already auto-vectorized, so no need to explicitly write a 4f version of SkRect::round()
Bug: skia:
Change-Id: I098945767bfcaa7093d770c376bd17ff3bdc9983
Reviewed-on: https://skia-review.googlesource.com/32060
Commit-Queue: Mike Reed <reed@google.com>
Reviewed-by: Florin Malita <fmalita@chromium.org>
diff --git a/tests/MathTest.cpp b/tests/MathTest.cpp
index 3d30f0d..bbd551e 100644
--- a/tests/MathTest.cpp
+++ b/tests/MathTest.cpp
@@ -671,3 +671,25 @@
test_nextsizepow2(reporter, SIZE_MAX, SIZE_MAX);
}
+
+DEF_TEST(FloatSaturate, reporter) {
+ const struct {
+ float fFloat;
+ int fExpectedInt;
+ } recs[] = {
+ { 0, 0 },
+ { 100.5f, 100 },
+ { (float)SK_MaxS32, SK_MaxS32FitsInFloat },
+ { (float)SK_MinS32, SK_MinS32FitsInFloat },
+ { SK_MaxS32 * 100.0f, SK_MaxS32FitsInFloat },
+ { SK_MinS32 * 100.0f, SK_MinS32FitsInFloat },
+ { SK_ScalarInfinity, SK_MaxS32FitsInFloat },
+ { SK_ScalarNegativeInfinity, SK_MinS32FitsInFloat },
+ { SK_ScalarNaN, SK_MaxS32FitsInFloat },
+ };
+
+ for (auto r : recs) {
+ int i = sk_float_saturate2int(r.fFloat);
+ REPORTER_ASSERT(reporter, r.fExpectedInt == i);
+ }
+}