diff options
author | rmistry@google.com <rmistry@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2012-08-23 18:14:13 +0000 |
---|---|---|
committer | rmistry@google.com <rmistry@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2012-08-23 18:14:13 +0000 |
commit | d6176b0dcacb124539e0cfd051e6d93a9782f020 (patch) | |
tree | 9e6f4b465e54c9b26e1ba70cd8890b55abb08464 /src/svg | |
parent | fbfcd5602128ec010c82cb733c9cdc0a3254f9f3 (diff) |
Result of running tools/sanitize_source_files.py (which was added in https://codereview.appspot.com/6465078/)
This CL is part II of IV (I broke down the 1280 files into 4 CLs).
Review URL: https://codereview.appspot.com/6474054
git-svn-id: http://skia.googlecode.com/svn/trunk@5263 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'src/svg')
-rw-r--r-- | src/svg/SkSVGClipPath.cpp | 2 | ||||
-rw-r--r-- | src/svg/SkSVGElements.cpp | 4 | ||||
-rw-r--r-- | src/svg/SkSVGGradient.cpp | 2 | ||||
-rw-r--r-- | src/svg/SkSVGPaintState.cpp | 12 | ||||
-rw-r--r-- | src/svg/SkSVGParser.cpp | 20 | ||||
-rw-r--r-- | src/svg/SkSVGPath.cpp | 2 | ||||
-rw-r--r-- | src/svg/SkSVGPolygon.cpp | 4 | ||||
-rw-r--r-- | src/svg/SkSVGPolygon.h | 2 | ||||
-rw-r--r-- | src/svg/SkSVGPolyline.cpp | 4 | ||||
-rw-r--r-- | src/svg/SkSVGPolyline.h | 2 | ||||
-rw-r--r-- | src/svg/SkSVGSVG.cpp | 6 | ||||
-rw-r--r-- | src/svg/SkSVGSVG.h | 4 | ||||
-rw-r--r-- | src/svg/SkSVGSymbol.cpp | 2 |
13 files changed, 33 insertions, 33 deletions
diff --git a/src/svg/SkSVGClipPath.cpp b/src/svg/SkSVGClipPath.cpp index 0a41764f84..fa3a79918d 100644 --- a/src/svg/SkSVGClipPath.cpp +++ b/src/svg/SkSVGClipPath.cpp @@ -32,7 +32,7 @@ void SkSVGClipPath::translate(SkSVGParser& parser, bool defState) { const char* refStr = &use->f_xlink_href.c_str()[1]; SkASSERT(parser.getIDs().find(refStr, &ref)); SkASSERT(ref); - if (ref->getType() == SkSVGType_Rect) + if (ref->getType() == SkSVGType_Rect) parser._addAttribute("rectangle", refStr); else parser._addAttribute("path", refStr); diff --git a/src/svg/SkSVGElements.cpp b/src/svg/SkSVGElements.cpp index 0096fc055e..66dcd4e9c9 100644 --- a/src/svg/SkSVGElements.cpp +++ b/src/svg/SkSVGElements.cpp @@ -13,7 +13,7 @@ SkSVGBase::~SkSVGBase() { } -void SkSVGBase::addAttribute(SkSVGParser& parser, int attrIndex, +void SkSVGBase::addAttribute(SkSVGParser& parser, int attrIndex, const char* attrValue, size_t attrLength) { SkString* first = (SkString*) ((char*) this + sizeof(SkSVGElement)); first += attrIndex; @@ -82,7 +82,7 @@ void SkSVGElement::setIsDef() { //} void SkSVGElement::write(SkSVGParser& , SkString& ) { - SkASSERT(0); + SkASSERT(0); } diff --git a/src/svg/SkSVGGradient.cpp b/src/svg/SkSVGGradient.cpp index 27964e7328..d5f76394eb 100644 --- a/src/svg/SkSVGGradient.cpp +++ b/src/svg/SkSVGGradient.cpp @@ -84,7 +84,7 @@ void SkSVGGradient::write(SkSVGParser& parser, SkString& baseColor) { parser.fSuppressPaint = true; SkString originalID(f_id); f_id.set("mask"); // write out gradient named given name + color (less initial #) - f_id.append(baseColor.c_str() + 1); + f_id.append(baseColor.c_str() + 1); SkString originalColors; for (SkSVGElement** ptr = fChildren.begin(); ptr < fChildren.end(); ptr++) { SkSVGStop* colorElement = (SkSVGStop*) *ptr; diff --git a/src/svg/SkSVGPaintState.cpp b/src/svg/SkSVGPaintState.cpp index 1a30cb8a24..db3090074c 100644 --- a/src/svg/SkSVGPaintState.cpp +++ b/src/svg/SkSVGPaintState.cpp @@ -49,7 +49,7 @@ SkString* SkSVGPaint::operator[](int index) { return result; } -void SkSVGPaint::addAttribute(SkSVGParser& parser, int attrIndex, +void SkSVGPaint::addAttribute(SkSVGParser& parser, int attrIndex, const char* attrValue, size_t attrLength) { SkString* attr = (*this)[attrIndex]; switch(attrIndex) { @@ -221,7 +221,7 @@ void SkSVGPaint::setSave(SkSVGParser& parser) { sum.setConcat(matrix, sum); continue; } - if ( index == kClipPath) + if ( index == kClipPath) *clips.insert(0) = lastAttr; } walking = walking->fNext; @@ -267,7 +267,7 @@ void SkSVGPaint::setSave(SkSVGParser& parser) { #endif } -bool SkSVGPaint::writeChangedAttributes(SkSVGParser& parser, +bool SkSVGPaint::writeChangedAttributes(SkSVGParser& parser, SkSVGPaint& current, bool* changed) { SkSVGPaint& lastState = parser.fLastFlush; for (int index = kInitial + 1; index < kTerminal; index++) { @@ -285,7 +285,7 @@ bool SkSVGPaint::writeChangedAttributes(SkSVGParser& parser, case kEnableBackground: break; case kFill: - if (topAttr->equals("none") == false && lastAttr->equals("none") == true) + if (topAttr->equals("none") == false && lastAttr->equals("none") == true) parser._addAttribute("stroke", "false"); goto fillStrokeAttrCommon; case kFillRule: @@ -307,7 +307,7 @@ bool SkSVGPaint::writeChangedAttributes(SkSVGParser& parser, case kStopOpacity: break; case kStroke: - if (topAttr->equals("none") == false && lastAttr->equals("none") == true) + if (topAttr->equals("none") == false && lastAttr->equals("none") == true) parser._addAttribute("stroke", "true"); fillStrokeAttrCommon: if (strncmp(attrValue, "url(", 4) == 0) { @@ -442,7 +442,7 @@ addColor: } return true; } - + void SkSVGPaint::Push(SkSVGPaint** head, SkSVGPaint* newRecord) { newRecord->fNext = *head; *head = newRecord; diff --git a/src/svg/SkSVGParser.cpp b/src/svg/SkSVGParser.cpp index c8712b4796..74ea0236e0 100644 --- a/src/svg/SkSVGParser.cpp +++ b/src/svg/SkSVGParser.cpp @@ -35,9 +35,9 @@ static int gGeneratedMatrixID = 0; -SkSVGParser::SkSVGParser(SkXMLParserError* errHandler) : - SkXMLParser(errHandler), - fHead(&fEmptyPaint), fIDs(256), +SkSVGParser::SkSVGParser(SkXMLParserError* errHandler) : + SkXMLParser(errHandler), + fHead(&fEmptyPaint), fIDs(256), fXMLWriter(&fStream), fCurrElement(NULL), fInSVG(false), fSuppressPaint(false) { fLastTransform.reset(); fEmptyPaint.f_fill.set("black"); @@ -72,7 +72,7 @@ int SkSVGParser::findAttribute(SkSVGBase* element, const char* attrValue, size_t result = 0; while (result < count) { if (strncmp(attributes->fName, attrValue, len) == 0 && strlen(attributes->fName) == len) { - SkASSERT(result == (attributes->fOffset - + SkASSERT(result == (attributes->fOffset - (isPaint ? sizeof(SkString) : sizeof(SkSVGElement))) / sizeof(SkString)); return result; } @@ -199,9 +199,9 @@ bool SkSVGParser::onStartElementLen(const char name[], size_t len) { SkSVGTypes type = GetType(name, len); // SkASSERT(type >= 0); if (type < 0) { - type = SkSVGType_G; + type = SkSVGType_G; // return true; - } + } SkSVGElement* parent = fParents.count() > 0 ? fParents.top() : NULL; SkSVGElement* element = CreateElement(type, parent); bool result = false; @@ -219,7 +219,7 @@ bool SkSVGParser::onStartElementLen(const char name[], size_t len) { bool SkSVGParser::onText(const char text[], int len) { if (fInSVG == false) return false; - SkSVGTypes type = fCurrElement->getType(); + SkSVGTypes type = fCurrElement->getType(); if (type != SkSVGType_Text && type != SkSVGType_Tspan) return false; SkSVGText* textElement = (SkSVGText*) fCurrElement; @@ -310,7 +310,7 @@ void SkSVGParser::translateMatrix(SkString& string, SkString* stringID) { string.append(elems[index], end - elems[index] + 1); } string.remove(string.size() - 1, 1); - string.append(",0,0,1]"); + string.append(",0,0,1]"); _addAttribute("matrix", string); _endElement(); // matrix } @@ -434,8 +434,8 @@ const SkSVGTypeName gSVGTypeNames[] = { const int kSVGTypeNamesSize = SK_ARRAY_COUNT(gSVGTypeNames); SkSVGTypes SkSVGParser::GetType(const char match[], size_t len ) { - int index = SkStrSearch(&gSVGTypeNames[0].fName, kSVGTypeNamesSize, match, + int index = SkStrSearch(&gSVGTypeNames[0].fName, kSVGTypeNamesSize, match, len, sizeof(gSVGTypeNames[0])); - return index >= 0 && index < kSVGTypeNamesSize ? gSVGTypeNames[index].fType : + return index >= 0 && index < kSVGTypeNamesSize ? gSVGTypeNames[index].fType : (SkSVGTypes) -1; } diff --git a/src/svg/SkSVGPath.cpp b/src/svg/SkSVGPath.cpp index 92f5b14887..ab18a65a7b 100644 --- a/src/svg/SkSVGPath.cpp +++ b/src/svg/SkSVGPath.cpp @@ -29,7 +29,7 @@ void SkSVGPath::translate(SkSVGParser& parser, bool defState) { } if (hasMultiplePaths) { SkString& fillRule = parser.getPaintLast(SkSVGPaint::kFillRule); - if (fillRule.size() > 0) + if (fillRule.size() > 0) parser._addAttribute("fillType", fillRule.equals("evenodd") ? "evenOdd" : "winding"); } SVG_ADD_ATTRIBUTE(d); diff --git a/src/svg/SkSVGPolygon.cpp b/src/svg/SkSVGPolygon.cpp index 97cf5e06b5..4b458db9e7 100644 --- a/src/svg/SkSVGPolygon.cpp +++ b/src/svg/SkSVGPolygon.cpp @@ -18,7 +18,7 @@ const SkSVGAttribute SkSVGPolygon::gAttributes[] = { DEFINE_SVG_INFO(Polygon) -void SkSVGPolygon::addAttribute(SkSVGParser& parser, int attrIndex, +void SkSVGPolygon::addAttribute(SkSVGParser& parser, int attrIndex, const char* attrValue, size_t attrLength) { INHERITED::addAttribute(parser, attrIndex, attrValue, attrLength); } @@ -27,7 +27,7 @@ void SkSVGPolygon::translate(SkSVGParser& parser, bool defState) { parser._startElement("polygon"); SkSVGElement::translate(parser, defState); SVG_ADD_ATTRIBUTE(points); - if (f_fillRule.size() > 0) + if (f_fillRule.size() > 0) parser._addAttribute("fillType", f_fillRule.equals("evenodd") ? "evenOdd" : "winding"); parser._endElement(); } diff --git a/src/svg/SkSVGPolygon.h b/src/svg/SkSVGPolygon.h index d353764896..b2848d064b 100644 --- a/src/svg/SkSVGPolygon.h +++ b/src/svg/SkSVGPolygon.h @@ -14,7 +14,7 @@ class SkSVGPolygon : public SkSVGPolyline { DECLARE_SVG_INFO(Polygon); - virtual void addAttribute(SkSVGParser& , int attrIndex, + virtual void addAttribute(SkSVGParser& , int attrIndex, const char* attrValue, size_t attrLength); private: typedef SkSVGPolyline INHERITED; diff --git a/src/svg/SkSVGPolyline.cpp b/src/svg/SkSVGPolyline.cpp index fe83c04655..83dad488a5 100644 --- a/src/svg/SkSVGPolyline.cpp +++ b/src/svg/SkSVGPolyline.cpp @@ -24,7 +24,7 @@ const SkSVGAttribute SkSVGPolyline::gAttributes[] = { DEFINE_SVG_INFO(Polyline) -void SkSVGPolyline::addAttribute(SkSVGParser& , int attrIndex, +void SkSVGPolyline::addAttribute(SkSVGParser& , int attrIndex, const char* attrValue, size_t attrLength) { if (attrIndex != kPoints) return; @@ -37,7 +37,7 @@ void SkSVGPolyline::translate(SkSVGParser& parser, bool defState) { parser._startElement("polyline"); INHERITED::translate(parser, defState); SVG_ADD_ATTRIBUTE(points); - if (f_fillRule.size() > 0) + if (f_fillRule.size() > 0) parser._addAttribute("fillType", f_fillRule.equals("evenodd") ? "evenOdd" : "winding"); parser._endElement(); } diff --git a/src/svg/SkSVGPolyline.h b/src/svg/SkSVGPolyline.h index 62dc417427..b8d5af45a8 100644 --- a/src/svg/SkSVGPolyline.h +++ b/src/svg/SkSVGPolyline.h @@ -15,7 +15,7 @@ class SkSVGPolyline : public SkSVGElement { DECLARE_SVG_INFO(Polyline); - virtual void addAttribute(SkSVGParser& , int attrIndex, + virtual void addAttribute(SkSVGParser& , int attrIndex, const char* attrValue, size_t attrLength); protected: SkString f_clipRule; diff --git a/src/svg/SkSVGSVG.cpp b/src/svg/SkSVGSVG.cpp index a072d09f69..fcce62de9e 100644 --- a/src/svg/SkSVGSVG.cpp +++ b/src/svg/SkSVGSVG.cpp @@ -48,8 +48,8 @@ void SkSVGSVG::translate(SkSVGParser& parser, bool defState) { box.fTop = SkScalarDiv(viewBox[1], height); box.fRight = SkScalarDiv(viewBox[2], width); box.fBottom = SkScalarDiv(viewBox[3], height); - if (box.fLeft == 0 && box.fTop == 0 && - box.fRight == SK_Scalar1 && box.fBottom == SK_Scalar1) + if (box.fLeft == 0 && box.fTop == 0 && + box.fRight == SK_Scalar1 && box.fBottom == SK_Scalar1) return; parser._startElement("matrix"); if (box.fLeft != 0) { @@ -72,5 +72,5 @@ void SkSVGSVG::translate(SkSVGParser& parser, bool defState) { y.appendScalar(box.fBottom); parser._addAttributeLen("scaleY", y.c_str(), y.size()); } - parser._endElement(); + parser._endElement(); } diff --git a/src/svg/SkSVGSVG.h b/src/svg/SkSVGSVG.h index 7d1e16ad41..155f9a9bab 100644 --- a/src/svg/SkSVGSVG.h +++ b/src/svg/SkSVGSVG.h @@ -22,11 +22,11 @@ private: SkString f_width; SkString f_version; SkString f_viewBox; - SkString f_x; + SkString f_x; SkString f_xml_space; SkString f_xmlns; SkString f_xml_xlink; - SkString f_y; + SkString f_y; typedef SkSVGElement INHERITED; }; diff --git a/src/svg/SkSVGSymbol.cpp b/src/svg/SkSVGSymbol.cpp index a98846719a..ce341e6492 100644 --- a/src/svg/SkSVGSymbol.cpp +++ b/src/svg/SkSVGSymbol.cpp @@ -18,5 +18,5 @@ DEFINE_SVG_INFO(Symbol) void SkSVGSymbol::translate(SkSVGParser& parser, bool defState) { INHERITED::translate(parser, defState); - // !!! children need to be written into document + // !!! children need to be written into document } |