Initial error handling code

src/core/SkFlattenableBuffers.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 "SkFlattenableBuffers.h"
 #include "SkPaint.h"
 #include "SkTypeface.h"
 
-SkFlattenableReadBuffer::SkFlattenableReadBuffer() {
+SkFlattenableReadBuffer::SkFlattenableReadBuffer() :
+    fError(false) {

[mtklein, 2013/10/07 19:29:56]

This is what makes me think fError doesn't really want to be on
SkFlattenableReadBuffer.  It doesn't use it.

     // Set default values. These should be explicitly set by our client
     // via setFlags() if the buffer came from serialization.
     fFlags = 0;
 #ifdef SK_SCALAR_IS_FLOAT
     fFlags |= kScalarIsFloat_Flag;
 #endif
     if (8 == sizeof(void*)) {
         fFlags |= kPtrIs64Bit_Flag;
     }
 }
@@ skipping 24 matching lines @@
     this->writeByteArray(ptrStorage, sizeof(void*));
 }
 
 void SkFlattenableWriteBuffer::writePaint(const SkPaint& paint) {
     paint.flatten(*this);
 }
 
 void SkFlattenableWriteBuffer::flattenObject(SkFlattenable* obj, SkFlattenableWriteBuffer& buffer) {
     obj->flatten(buffer);
 }