Sanitizing source files in Skia_Periodic_House_Keeping
authorskia.committer@gmail.com <skia.committer@gmail.com@2bbb7eff-a529-9590-31e7-b0007b416f81>
Fri, 30 Nov 2012 02:01:24 +0000 (02:01 +0000)
committerskia.committer@gmail.com <skia.committer@gmail.com@2bbb7eff-a529-9590-31e7-b0007b416f81>
Fri, 30 Nov 2012 02:01:24 +0000 (02:01 +0000)
git-svn-id: http://skia.googlecode.com/svn/trunk@6632 2bbb7eff-a529-9590-31e7-b0007b416f81

experimental/Intersection/Simplify.cpp
include/core/SkClipStack.h
include/core/SkRRect.h
src/core/SkDraw.cpp
src/core/SkRRect.cpp
src/utils/SkPictureUtils.cpp
tests/PictureTest.cpp
tests/RoundRectTest.cpp
tools/PictureRenderer.cpp

index a221e6b61462614b148087a36f25351d1427f828..c52ff5f857113ae26e3b09f06a487c9f28a6563a 100644 (file)
@@ -1092,10 +1092,10 @@ public:
         return activeOp(xorMiMask, xorSuMask, index, endIndex, op, sumMiWinding, sumSuWinding,
             maxWinding, sumWinding, oppMaxWinding, oppSumWinding);
     }
-    
+
     bool activeOp(int xorMiMask, int xorSuMask,
             int index, int endIndex, ShapeOp op,
-            int& sumMiWinding, int& sumSuWinding, 
+            int& sumMiWinding, int& sumSuWinding,
             int& maxWinding, int& sumWinding, int& oppMaxWinding, int& oppSumWinding) {
         setUpWindings(index, endIndex, sumMiWinding, sumSuWinding,
                 maxWinding, sumWinding, oppMaxWinding, oppSumWinding);
@@ -1963,11 +1963,11 @@ public:
      The Opp variable name part designates that the value is for the Opposite operator.
      Opposite values result from combining coincident spans.
      */
-     
+
     Segment* findNextOp(SkTDArray<Span*>& chase, int& nextStart, int& nextEnd,
             bool& unsortable, ShapeOp op, const int xorMiMask, const int xorSuMask) {
         const int startIndex = nextStart;
-        const int endIndex = nextEnd;    
+        const int endIndex = nextEnd;
         SkASSERT(startIndex != endIndex);
         const int count = fTs.count();
         SkASSERT(startIndex < endIndex ? startIndex < count - 1 : startIndex > 0);
@@ -2803,7 +2803,7 @@ public:
         markDoneBinary(SkMin32(index, endIndex), winding, oppWinding);
         return last;
     }
-    
+
     Span* markAndChaseDoneBinary(int index, int endIndex) {
         int step = SkSign32(endIndex - index);
         Span* last = innerChaseDoneBinary(index, step);
@@ -2820,7 +2820,7 @@ public:
         markWinding(min, winding);
         return last;
     }
-    
+
     Span* markAndChaseWinding(int index, int endIndex, int winding, int oppWinding) {
         int min = SkMin32(index, endIndex);
         int step = SkSign32(endIndex - index);
@@ -3262,7 +3262,7 @@ public:
         }
         return oppWinding;
     }
-    
+
     int updateOppWinding(const Angle* angle) const {
         int startIndex = angle->start();
         int endIndex = angle->end();
index 4155bd0062bcd9da662ab31cfddd6c65d46595ad..d4f6a5298589c8df25d9e033f03924247f2178a2 100644 (file)
@@ -103,7 +103,7 @@ public:
            clips are Booleaned together. */
         SkClipStack::BoundsType fFiniteBoundType;
         SkRect                  fFiniteBound;
+
         // When element is applied to the previous elements in the stack is the result known to be
         // equivalent to a single rect intersection? IIOW, is the clip effectively a rectangle.
         bool                    fIsIntersectionOfRects;
@@ -163,7 +163,7 @@ public:
         inline bool operator!=(const Element& b) const;
         inline bool canBeIntersectedInPlace(int saveCount, SkRegion::Op op) const;
         /* This method checks to see if two rect clips can be safely merged into one. The issue here
-          is that to be strictly correct all the edges of the resulting rect must have the same 
+          is that to be strictly correct all the edges of the resulting rect must have the same
           anti-aliasing. */
         bool rectRectIntersectAllowed(const SkRect& newR, bool newAA) const;
         /** Determines possible finite bounds for the Element given the previous element of the
index 046df36c200c74eda7d7cbd967a1fbe22092abff..66c1ee24da058d667eb4615a8cf75c3b552e0e2c 100644 (file)
 //      add RR rendering shader to Ganesh (akin to cicle drawing code)
 //          - only for simple RRs
 //      detect and triangulate RRectorii rather than falling back to SW in Ganesh
-//      
+//
 
 /** \class SkRRect
 
-    The SkRRect class represents a rounded rect with a potentially different 
-    radii for each corner. It does not have a constructor so must be 
-    initialized with one of the initialization functions (e.g., setEmpty, 
+    The SkRRect class represents a rounded rect with a potentially different
+    radii for each corner. It does not have a constructor so must be
+    initialized with one of the initialization functions (e.g., setEmpty,
     setRectRadii, etc.)
 
-    This class is intended to roughly match CSS' border-*-*-radius capabilities. 
+    This class is intended to roughly match CSS' border-*-*-radius capabilities.
     This means:
-        If either of a corner's radii are 0 the corner will be square. 
+        If either of a corner's radii are 0 the corner will be square.
         Negative radii are not allowed (they are clamped to zero).
         If the corner curves overlap they will be proportionally reduced to fit.
 */
 class SK_API SkRRect {
 public:
-    /** 
+    /**
      * Enum to capture the various possible subtypes of RR. Accessed
      * by type(). The subtypes become progressively less restrictive.
      */
@@ -60,16 +60,16 @@ public:
 
         //!< The RR is actually a (non-empty) rect (i.e., at least one radius
         //!< at each corner is zero)
-        kRect_Type,  
+        kRect_Type,
 
-        //!< The RR is actually a (non-empty) oval (i.e., all x radii are equal 
+        //!< The RR is actually a (non-empty) oval (i.e., all x radii are equal
         //!< and >= width/2 and all the y radii are equal and >= height/2
-        kOval_Type,  
+        kOval_Type,
 
-        //!< The RR is non-empty and all the x radii are equal & all y radii 
-        //!< are equal but it is not an oval (i.e., there are lines between 
+        //!< The RR is non-empty and all the x radii are equal & all y radii
+        //!< are equal but it is not an oval (i.e., there are lines between
         //!< the curves) nor a rect (i.e., both radii are non-zero)
-        kSimple_Type,  
+        kSimple_Type,
 
         //!< A fully general (non-empty) RR. Some of the x and/or y radii are
         //!< different from the others and there must be one corner where
@@ -77,7 +77,7 @@ public:
         kComplex_Type,
     };
 
-    /** 
+    /**
      * Returns the RR's sub type.
      */
     Type type() const {
@@ -90,10 +90,10 @@ public:
         return fType;
     }
 
-    /** 
+    /**
      * Set this RR to the empty rectangle (0,0,0,0) with 0 x & y radii.
      */
-    void setEmpty() { 
+    void setEmpty() {
         fRect.setEmpty();
         memset(fRadii, 0, sizeof(fRadii));
         fType = kEmpty_Type;
@@ -101,7 +101,7 @@ public:
         SkDEBUGCODE(this->validate();)
     }
 
-    /** 
+    /**
      * Set this RR to match the supplied rect. All radii will be 0.
      */
     void setRect(const SkRect& rect) {
@@ -117,7 +117,7 @@ public:
         SkDEBUGCODE(this->validate();)
     }
 
-    /** 
+    /**
      * Set this RR to match the supplied oval. All x radii will equal half the
      * width and all y radii will equal half the height.
      */
@@ -139,17 +139,17 @@ public:
         SkDEBUGCODE(this->validate();)
     }
 
-    /** 
+    /**
      * Initialize the RR with the same radii for all four corners.
      */
     void setRectXY(const SkRect& rect, SkScalar xRad, SkScalar yRad);
 
-    /** 
+    /**
      * Initialize the RR with potentially different radii for all four corners.
      */
     void setRectRadii(const SkRect& rect, const SkVector radii[4]);
 
-    // The radii are stored in UL, UR, LR, LL order. 
+    // The radii are stored in UL, UR, LR, LL order.
     enum Corner {
         kUpperLeft_Corner,
         kUpperRight_Corner,
@@ -204,7 +204,7 @@ private:
 
     SkRect fRect;
     // Radii order is UL, UR, LR, LL. Use Corner enum to index into fRadii[]
-    SkVector fRadii[4];     
+    SkVector fRadii[4];
     mutable Type fType;
     // TODO: add padding so we can use memcpy for flattening and not copy
     // uninitialized data
index 46b10b70c6fa46ec1b682034410c8a1c54b3d2fb..e1ddb337f0fe65d9fb89f150d6c2c625eeeef774 100644 (file)
@@ -334,10 +334,10 @@ static void bw_pt_rect_16_hair_proc(const PtProcRec& rec,
     uint32_t value;
     const SkBitmap* bitmap = blitter->justAnOpaqueColor(&value);
     SkASSERT(bitmap);
-    
+
     uint16_t* addr = bitmap->getAddr16(0, 0);
     int rb = bitmap->rowBytes();
-    
+
     for (int i = 0; i < count; i++) {
         int x = SkScalarFloorToInt(devPts[i].fX);
         int y = SkScalarFloorToInt(devPts[i].fY);
@@ -355,10 +355,10 @@ static void bw_pt_rect_32_hair_proc(const PtProcRec& rec,
     uint32_t value;
     const SkBitmap* bitmap = blitter->justAnOpaqueColor(&value);
     SkASSERT(bitmap);
-    
+
     SkPMColor* addr = bitmap->getAddr32(0, 0);
     int rb = bitmap->rowBytes();
-    
+
     for (int i = 0; i < count; i++) {
         int x = SkScalarFloorToInt(devPts[i].fX);
         int y = SkScalarFloorToInt(devPts[i].fY);
@@ -417,13 +417,13 @@ static void bw_square_proc(const PtProcRec& rec, const SkPoint devPts[],
     for (int i = 0; i < count; i++) {
         SkFixed x = SkScalarToFixed(devPts[i].fX);
         SkFixed y = SkScalarToFixed(devPts[i].fY);
-        
+
         SkXRect r;
         r.fLeft = x - radius;
         r.fTop = y - radius;
         r.fRight = x + radius;
         r.fBottom = y + radius;
-        
+
         SkScan::FillXRect(r, *rec.fRC, blitter);
     }
 }
index 03b8bbb9308f99a7e3a43118fce7938f79d34ce8..db1c7dd965b931eaeaf7b2df827d06b7f84bc38c 100644 (file)
@@ -75,14 +75,14 @@ void SkRRect::setRectRadii(const SkRect& rect, const SkVector radii[4]) {
     }
 
     // Proportionally scale down all radii to fit. Find the minimum ratio
-    // of a side and the radii on that side (for all four sides) and use 
+    // of a side and the radii on that side (for all four sides) and use
     // that to scale down _all_ the radii. This algorithm is from the
     // W3 spec (http://www.w3.org/TR/css3-background/) section 5.5 - Overlapping
     // Curves:
-    // "Let f = min(Li/Si), where i is one of { top, right, bottom, left }, 
+    // "Let f = min(Li/Si), where i is one of { top, right, bottom, left },
     //   Si is the sum of the two corresponding radii of the corners on side i,
-    //   and Ltop = Lbottom = the width of the box, 
-    //   and Lleft = Lright = the height of the box. 
+    //   and Ltop = Lbottom = the width of the box,
+    //   and Lleft = Lright = the height of the box.
     // If f < 1, then all corner radii are reduced by multiplying them by f."
     SkScalar scale = SK_Scalar1;
 
@@ -143,7 +143,7 @@ bool SkRRect::contains(SkScalar x, SkScalar y) const {
         canonicalPt.set(x - fRect.centerX(), y - fRect.centerY());
         index = kUpperLeft_Corner;  // any corner will do in this case
     } else {
-        if (x < fRect.fLeft + fRadii[kUpperLeft_Corner].fX && 
+        if (x < fRect.fLeft + fRadii[kUpperLeft_Corner].fX &&
             y < fRect.fTop + fRadii[kUpperLeft_Corner].fY) {
             // UL corner
             index = kUpperLeft_Corner;
@@ -252,7 +252,7 @@ void SkRRect::validate() const {
             SkASSERT(fRect.isEmpty());
             SkASSERT(allRadiiZero && allRadiiSame && allCornersSquare);
 
-            SkASSERT(0 == fRect.fLeft && 0 == fRect.fTop && 
+            SkASSERT(0 == fRect.fLeft && 0 == fRect.fTop &&
                      0 == fRect.fRight && 0 == fRect.fBottom);
             break;
         case kRect_Type:
index 16cf11d2e06c2940005ceb366dff12ddd28d6a3c..46ca6d6f9e7f394a348913585703f8675752552f 100644 (file)
@@ -76,7 +76,7 @@ public:
                              SkCanvas::Config8888 config8888) SK_OVERRIDE {
         not_supported();
     }
-    
+
     virtual void drawPaint(const SkDraw&, const SkPaint& paint) SK_OVERRIDE {
         this->addBitmapFromPaint(paint);
     }
@@ -146,7 +146,7 @@ protected:
 class NoSaveLayerCanvas : public SkCanvas {
 public:
     NoSaveLayerCanvas(SkDevice* device) : INHERITED(device) {}
-    
+
     // turn saveLayer() into save() for speed, should not affect correctness.
     virtual int saveLayer(const SkRect* bounds, const SkPaint* paint,
                           SaveFlags flags) SK_OVERRIDE {
index 567a8d200687f7a96a460e8c9039d2b176fe9aad..79ca3132f718f147a0bf34c0700fed58c5096405 100644 (file)
@@ -207,7 +207,7 @@ static void test_gatherpixelrefs(skiatest::Reporter* reporter) {
             SkAutoDataUnref adu(data);
 
             gather_from_colors(result, refs, N, &array);
-            
+
             /*
              *  GatherPixelRefs is conservative, so it can return more bitmaps
              *  that we actually can see (usually because of conservative bounds
index fd4780c6873cbb9e3b0061b5934d59325e1bcccd..a358c5a66f8b7983872ff2315267c840f839eb48 100644 (file)
@@ -16,7 +16,7 @@ static void test_round_rect_basic(skiatest::Reporter* reporter) {
     // Test out initialization methods
     SkPoint zeroPt = { 0.0, 0.0 };
     SkRRect empty;
-    
+
     empty.setEmpty();
 
     REPORTER_ASSERT(reporter, SkRRect::kEmpty_Type == empty.type());
@@ -48,7 +48,7 @@ static void test_round_rect_basic(skiatest::Reporter* reporter) {
     REPORTER_ASSERT(reporter, rr2.rect() == rect);
 
     for (int i = 0; i < 4; ++i) {
-        REPORTER_ASSERT(reporter, 
+        REPORTER_ASSERT(reporter,
                         rr2.radii((SkRRect::Corner) i).equalsWithinTolerance(halfPoint));
     }
 
@@ -63,7 +63,7 @@ static void test_round_rect_basic(skiatest::Reporter* reporter) {
     for (int i = 0; i < 4; ++i) {
         REPORTER_ASSERT(reporter, p == rr3.radii((SkRRect::Corner) i));
     }
-    
+
     //----
     SkPoint radii[4] = { { 5, 5 }, { 5, 5 }, { 5, 5 }, { 5, 5 } };
 
@@ -99,7 +99,7 @@ static void test_round_rect_basic(skiatest::Reporter* reporter) {
 // Test out the cases when the RR degenerates to a rect
 static void test_round_rect_rects(skiatest::Reporter* reporter) {
     SkRect r;
-    static const SkPoint pts[] = { 
+    static const SkPoint pts[] = {
         // Upper Left
         { -SK_Scalar1, -SK_Scalar1 },               // out
         { SK_Scalar1, SK_Scalar1 },                 // in
@@ -121,7 +121,7 @@ static void test_round_rect_rects(skiatest::Reporter* reporter) {
 
     //----
     SkRRect empty;
-    
+
     empty.setEmpty();
 
     REPORTER_ASSERT(reporter, SkRRect::kEmpty_Type == empty.type());
@@ -166,7 +166,7 @@ static void test_round_rect_ovals(skiatest::Reporter* reporter) {
     static const SkScalar kEps = 0.1f;
     static const SkScalar kWidthTol = SkScalarHalf(kWidth) * (SK_Scalar1 - SK_ScalarRoot2Over2);
     static const SkScalar kHeightTol = SkScalarHalf(kHeight) * (SK_Scalar1 - SK_ScalarRoot2Over2);
-    static const SkPoint pts[] = { 
+    static const SkPoint pts[] = {
         // Upper Left
         { kWidthTol - kEps, kHeightTol - kEps },       // out
         { kWidthTol + kEps, kHeightTol + kEps },       // in
@@ -204,7 +204,7 @@ static void test_round_rect_ovals(skiatest::Reporter* reporter) {
 static void test_round_rect_general(skiatest::Reporter* reporter) {
     static const SkScalar kEps = 0.1f;
     static const SkScalar kDist20 = 20 * (SK_Scalar1 - SK_ScalarRoot2Over2);
-    static const SkPoint pts[] = { 
+    static const SkPoint pts[] = {
         // Upper Left
         { kDist20 - kEps, kDist20 - kEps },       // out
         { kDist20 + kEps, kDist20 + kEps },       // in
@@ -236,7 +236,7 @@ static void test_round_rect_general(skiatest::Reporter* reporter) {
 
     //----
     static const SkScalar kDist50 = 50*(SK_Scalar1 - SK_ScalarRoot2Over2);
-    static const SkPoint pts2[] = { 
+    static const SkPoint pts2[] = {
         // Upper Left
         { -SK_Scalar1, -SK_Scalar1 },           // out
         { SK_Scalar1, SK_Scalar1 },             // in
index 69ce6f86fa2da57a55397def7c6c9967548a8773..3f96e909625f0cb2d5a5d00bda3f38b97d978728 100644 (file)
@@ -669,7 +669,7 @@ public:
                                        SkIntToScalar(fPicture->height()));
         SkData* data = SkPictureUtils::GatherPixelRefs(fPicture, bounds);
         SkSafeUnref(data);
-        
+
         return NULL == path;    // we don't have anything to write
     }
 
@@ -682,5 +682,5 @@ private:
 PictureRenderer* CreateGatherPixelRefsRenderer() {
     return SkNEW(GatherRenderer);
 }
-    
+
 } // namespace sk_tools