From c490f801b063a0837501feab3d12b73d71f46312 Mon Sep 17 00:00:00 2001 From: "jvanverth@google.com" Date: Mon, 4 Mar 2013 13:56:38 +0000 Subject: Change random number generator for 'tests' to SkMWCRandom. Also removes some unused headers and fixes a couple of bugs exposed by changing the random number generator: First, the function SkMatrix::getMaxStretch() had an error where it was testing the square of a number against near-zero. This led to it occasionally taking a cheaper but imprecise path for computing the eigenvalues of the matrix. It's been replaced with a check against the square of SK_ScalarNearlyZero. The second case was a failure in ClipStackTest, where it hit the rare case of a practically empty clip stack (it has a single Union) and we set a tight bounds. The bounds rect doesn't get set by GrReducedClip::ReduceClipStack() in this case, so when it clips the reduced stack it's clipping against garbage, and the resulting regions don't match. The solution is to initialize the tightBounds rect. git-svn-id: http://skia.googlecode.com/svn/trunk@7952 2bbb7eff-a529-9590-31e7-b0007b416f81 --- tests/SortTest.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'tests/SortTest.cpp') diff --git a/tests/SortTest.cpp b/tests/SortTest.cpp index 28c6e682de..6bfb300a72 100644 --- a/tests/SortTest.cpp +++ b/tests/SortTest.cpp @@ -15,7 +15,7 @@ extern "C" { } } -static void rand_array(SkRandom& rand, int array[], int n) { +static void rand_array(SkMWCRandom& rand, int array[], int n) { for (int j = 0; j < n; j++) { array[j] = rand.nextS() & 0xFF; } @@ -40,7 +40,7 @@ static void TestSort(skiatest::Reporter* reporter) { /** The random numbers are copied into this array, sorted by an SkSort, then this array is compared against the reference sort. */ int workingArray[SK_ARRAY_COUNT(randomArray)]; - SkRandom rand; + SkMWCRandom rand; for (int i = 0; i < 10000; i++) { int count = rand.nextRangeU(1, SK_ARRAY_COUNT(randomArray)); -- cgit v1.2.3