diff options
author | 2014-02-11 11:18:12 +0000 | |
---|---|---|
committer | 2014-02-11 11:18:12 +0000 | |
commit | afe38b16f082c8d88039339591376be54c7e801b (patch) | |
tree | 9bd995c91958e9fcd4f2d6cb155cc5aede6e4b63 | |
parent | 046f1f6ff4b2b3f4571a9562e74f41e82419a4a1 (diff) |
Fix broken debug build
BUG=skia:
R=tomhudson@google.com
Author: iancottrell@google.com
Review URL: https://codereview.chromium.org/143223005
git-svn-id: http://skia.googlecode.com/svn/trunk@13397 2bbb7eff-a529-9590-31e7-b0007b416f81
-rw-r--r-- | src/core/SkWriter32.cpp | 4 | ||||
-rw-r--r-- | tests/Writer32Test.cpp | 14 |
2 files changed, 5 insertions, 13 deletions
diff --git a/src/core/SkWriter32.cpp b/src/core/SkWriter32.cpp index 7285459c3d..1e8a10c87c 100644 --- a/src/core/SkWriter32.cpp +++ b/src/core/SkWriter32.cpp @@ -81,6 +81,6 @@ void SkWriter32::growToAtLeast(size_t size) { // Expand the array so all reserved space is "used", we maintain the // amount we have written manually outside the array fInternal.setCount(fCapacity); - SkASSERT(fInternal.count() == fCapacity); - SkASSERT(fInternal.reserved() == fCapacity); + SkASSERT(fInternal.count() == (int)fCapacity); + SkASSERT(fInternal.reserved() == (int)fCapacity); } diff --git a/tests/Writer32Test.cpp b/tests/Writer32Test.cpp index 2f8b5c6edf..4d421c8060 100644 --- a/tests/Writer32Test.cpp +++ b/tests/Writer32Test.cpp @@ -203,17 +203,9 @@ DEF_TEST(Writer32_contiguous, reporter) { test1(reporter, &writer); REPORTER_ASSERT(reporter, writer.contiguousArray() != NULL); - // This write is too big for the 32 byte storage block we provide. - writer.reset(storage, 32); - test2(reporter, &writer); - // Some data is in storage, some in writer's internal storage. - REPORTER_ASSERT(reporter, writer.contiguousArray() == NULL); - - writer.reset(); - test2(reporter, &writer); - // There is no external storage. All the data is in internal storage, - // so we can always read it contiguously. - REPORTER_ASSERT(reporter, writer.contiguousArray() != NULL); + // Everything other aspect of contiguous/non-contiguous is an + // implementation detail, not part of the public contract for + // SkWriter32, and so not tested here. } DEF_TEST(Writer32_small, reporter) { |