From 20eee3f047f56b7715b311313b2998daaaf08a96 Mon Sep 17 00:00:00 2001 From: robertphillips Date: Fri, 19 Jun 2015 05:14:26 -0700 Subject: Added check for ill-conditioned invert sk_inv_determinant has a guard that the determinant can't get too big so this CL only checks if the determinant gets too small. BUG=492263 Review URL: https://codereview.chromium.org/1188433011 --- tests/MatrixTest.cpp | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'tests/MatrixTest.cpp') diff --git a/tests/MatrixTest.cpp b/tests/MatrixTest.cpp index 72e0dcc021..2296b8f2b5 100644 --- a/tests/MatrixTest.cpp +++ b/tests/MatrixTest.cpp @@ -838,6 +838,13 @@ DEF_TEST(Matrix, reporter) { REPORTER_ASSERT(reporter, !mat.invert(NULL)); REPORTER_ASSERT(reporter, !mat.invert(&inverse)); + // Inverting this matrix results in a non-finite matrix + mat.setAll(0.0f, 1.0f, 2.0f, + 0.0f, 1.0f, -3.40277175e+38f, + 1.00003040f, 1.0f, 0.0f); + REPORTER_ASSERT(reporter, !mat.invert(NULL)); + REPORTER_ASSERT(reporter, !mat.invert(&inverse)); + // rectStaysRect test { static const struct { -- cgit v1.2.3