Adding size parameter to read array functions

src/core/SkOrderedReadBuffer.cpp

 
 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkBitmap.h"
 #include "SkErrorInternals.h"
@@ skipping 119 matching lines @@
 }
 
 void SkOrderedReadBuffer::readRegion(SkRegion* region) {
     fReader.readRegion(region);
 }
 
 void SkOrderedReadBuffer::readPath(SkPath* path) {
     fReader.readPath(path);
 }
 
-uint32_t SkOrderedReadBuffer::readByteArray(void* value) {
-    const uint32_t length = fReader.readU32();
-    memcpy(value, fReader.skip(SkAlign4(length)), length);
-    return length;
+bool SkOrderedReadBuffer::readByteArray(void* value, size_t size) {
+    const size_t length = this->getArrayCount();
+    if (length == size) {
+        (void)fReader.skip(sizeof(uint32_t)); // Skip array count
+        memcpy(value, fReader.skip(SkAlign4(length)), length);
+        return true;
+    }
+    SkASSERT(false);
+    return false;
 }
 
-uint32_t SkOrderedReadBuffer::readColorArray(SkColor* colors) {
-    const uint32_t count = fReader.readU32();
-    const uint32_t byteLength = count * sizeof(SkColor);
-    memcpy(colors, fReader.skip(SkAlign4(byteLength)), byteLength);
-    return count;
+bool SkOrderedReadBuffer::readColorArray(SkColor* colors, size_t size) {
+    const uint32_t count = this->getArrayCount();
+    const size_t byteLength = count * sizeof(SkColor);
+    if (byteLength == size) {
+        (void)fReader.skip(sizeof(uint32_t)); // Skip array count
+        memcpy(colors, fReader.skip(SkAlign4(byteLength)), byteLength);
+        return true;
+    }
+    SkASSERT(false);
+    return false;
 }
 
-uint32_t SkOrderedReadBuffer::readIntArray(int32_t* values) {
-    const uint32_t count = fReader.readU32();
-    const uint32_t byteLength = count * sizeof(int32_t);
-    memcpy(values, fReader.skip(SkAlign4(byteLength)), byteLength);
-    return count;
+bool SkOrderedReadBuffer::readIntArray(int32_t* values, size_t size) {
+    const uint32_t count = this->getArrayCount();
+    const size_t byteLength = count * sizeof(int32_t);
+    if (byteLength == size) {
+        (void)fReader.skip(sizeof(uint32_t)); // Skip array count
+        memcpy(values, fReader.skip(SkAlign4(byteLength)), byteLength);
+        return true;
+    }
+    SkASSERT(false);
+    return false;
 }
 
-uint32_t SkOrderedReadBuffer::readPointArray(SkPoint* points) {
-    const uint32_t count = fReader.readU32();
-    const uint32_t byteLength = count * sizeof(SkPoint);
-    memcpy(points, fReader.skip(SkAlign4(byteLength)), byteLength);
-    return count;
+bool SkOrderedReadBuffer::readPointArray(SkPoint* points, size_t size) {
+    const uint32_t count = this->getArrayCount();
+    const size_t byteLength = count * sizeof(SkPoint);
+    if (byteLength == size) {
+        (void)fReader.skip(sizeof(uint32_t)); // Skip array count
+        memcpy(points, fReader.skip(SkAlign4(byteLength)), byteLength);
+        return true;
+    }
+    SkASSERT(false);
+    return false;
 }
 
-uint32_t SkOrderedReadBuffer::readScalarArray(SkScalar* values) {
-    const uint32_t count = fReader.readU32();
-    const uint32_t byteLength = count * sizeof(SkScalar);
-    memcpy(values, fReader.skip(SkAlign4(byteLength)), byteLength);
-    return count;
+bool SkOrderedReadBuffer::readScalarArray(SkScalar* values, size_t size) {
+    const uint32_t count = this->getArrayCount();
+    const size_t byteLength = count * sizeof(SkScalar);
+    if (byteLength == size) {
+        (void)fReader.skip(sizeof(uint32_t)); // Skip array count
+        memcpy(values, fReader.skip(SkAlign4(byteLength)), byteLength);
+        return true;
+    }
+    SkASSERT(false);
+    return false;
 }
 
 uint32_t SkOrderedReadBuffer::getArrayCount() {
     return *(uint32_t*)fReader.peek();
 }
 
 void SkOrderedReadBuffer::readBitmap(SkBitmap* bitmap) {
     const int width = this->readInt();
     const int height = this->readInt();
     // The writer stored a boolean value to determine whether an SkBitmapHeap was used during
@@ skipping 127 matching lines @@
         if (sizeRecorded != sizeRead) {
             // we could try to fix up the offset...
             sk_throw();
         }
     } else {
         // we must skip the remaining data
         fReader.skip(sizeRecorded);
     }
     return obj;
 }