Initial error handling code

include/core/SkRect.h

 
 /*
  * Copyright 2006 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #ifndef SkRect_DEFINED
@@ skipping 80 matching lines @@
      *  method is defined to return (bottom + top) >> 1
      *
      *  This is a specific "truncation" of the average, which is different than
      *  (bottom + top) / 2 when the sum is negative.
      */
     int centerY() const { return (fBottom + fTop) >> 1; }
 
     /**
      *  Return true if the rectangle's width or height are <= 0
      */
     bool isEmpty() const { return fLeft >= fRight || fTop >= fBottom; }

[reed1, 2013/09/05 19:06:39]

Separate note: given our recent realization about overflow even in ints, perhaps
we should change isEmpty and isInverted...

isEmpty : return fRight - fLeft <= 0
isInverted : return fRight - fLeft < 0

We can think about that after this CL, just wanted to mention it.

 
+    bool isInverted() const { return fLeft > fRight || fTop > fBottom; }
+
     bool isLargest() const { return SK_MinS32 == fLeft &&
                                     SK_MinS32 == fTop &&
                                     SK_MaxS32 == fRight &&
                                     SK_MaxS32 == fBottom; }
 
     friend bool operator==(const SkIRect& a, const SkIRect& b) {
         return !memcmp(&a, &b, sizeof(a));
     }
 
     friend bool operator!=(const SkIRect& a, const SkIRect& b) {
@@ skipping 299 matching lines @@
               SkIntToScalar(irect.fRight),
               SkIntToScalar(irect.fBottom));
         return r;
     }
 
     /**
      *  Return true if the rectangle's width or height are <= 0
      */
     bool isEmpty() const { return fLeft >= fRight || fTop >= fBottom; }
 
+    bool isInverted() const { return fLeft > fRight || fTop > fBottom; }
+
     /**
      *  Returns true iff all values in the rect are finite. If any are
      *  infinite or NaN (or SK_FixedNaN when SkScalar is fixed) then this
      *  returns false.
      */
     bool isFinite() const {
 #ifdef SK_SCALAR_IS_FLOAT
         float accum = 0;
         accum *= fLeft;
         accum *= fTop;
@@ skipping 365 matching lines @@
      */
     void sort();
 
     /**
      *  cast-safe way to treat the rect as an array of (4) SkScalars.
      */
     const SkScalar* asScalars() const { return &fLeft; }
 };
 
 #endif