diff options
author | sugoi@google.com <sugoi@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2013-02-27 19:17:41 +0000 |
---|---|---|
committer | sugoi@google.com <sugoi@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2013-02-27 19:17:41 +0000 |
commit | 54f0d1b7113cb0dc184e522539aab1030a28a421 (patch) | |
tree | 034f4823f8b9ed4ca84d28cbaa08de582edea4c5 /tests/MemsetTest.cpp | |
parent | 4e0d96b6558f0360993e3ecbe0975017a195d4f2 (diff) |
Tests : Unused parameters cleanup
I removed unused parameters in the tests wherever it was trivial to do so. I'm trying to get the easy ones out of the way before we get into more involved discussions around this.
Review URL: https://codereview.appspot.com/7394055
git-svn-id: http://skia.googlecode.com/svn/trunk@7891 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'tests/MemsetTest.cpp')
-rw-r--r-- | tests/MemsetTest.cpp | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/tests/MemsetTest.cpp b/tests/MemsetTest.cpp index ad1a21c77c..c79915989d 100644 --- a/tests/MemsetTest.cpp +++ b/tests/MemsetTest.cpp @@ -79,9 +79,10 @@ static void test_16(skiatest::Reporter* reporter) { uint16_t* base = &buffer[PAD + alignment]; sk_memset16(base, VALUE16, count); - compare16(buffer, 0, PAD + alignment); - compare16(base, VALUE16, count); - compare16(base + count, 0, TOTAL - count - PAD - alignment); + REPORTER_ASSERT(reporter, + compare16(buffer, 0, PAD + alignment) && + compare16(base, VALUE16, count) && + compare16(base + count, 0, TOTAL - count - PAD - alignment)); } } } @@ -96,9 +97,10 @@ static void test_32(skiatest::Reporter* reporter) { uint32_t* base = &buffer[PAD + alignment]; sk_memset32(base, VALUE32, count); - compare32(buffer, 0, PAD + alignment); - compare32(base, VALUE32, count); - compare32(base + count, 0, TOTAL - count - PAD - alignment); + REPORTER_ASSERT(reporter, + compare32(buffer, 0, PAD + alignment) && + compare32(base, VALUE32, count) && + compare32(base + count, 0, TOTAL - count - PAD - alignment)); } } } |