Reviewed by Mitz.
authorrwlbuis <rwlbuis@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 10 Oct 2006 08:42:18 +0000 (08:42 +0000)
committerrwlbuis <rwlbuis@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 10 Oct 2006 08:42:18 +0000 (08:42 +0000)
        http://bugs.webkit.org/show_bug.cgi?id=11217
        Cleanup svg coding style

        Cleanup coding style in svg animation classes.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@16963 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 files changed:
WebCore/ChangeLog
WebCore/ksvg2/svg/SVGAnimateColorElement.cpp
WebCore/ksvg2/svg/SVGAnimateColorElement.h
WebCore/ksvg2/svg/SVGAnimateElement.cpp
WebCore/ksvg2/svg/SVGAnimateTransformElement.cpp
WebCore/ksvg2/svg/SVGAnimateTransformElement.h
WebCore/ksvg2/svg/SVGAnimatedPathData.cpp
WebCore/ksvg2/svg/SVGAnimatedPathData.h
WebCore/ksvg2/svg/SVGAnimatedPoints.cpp
WebCore/ksvg2/svg/SVGAnimatedPoints.h
WebCore/ksvg2/svg/SVGAnimationElement.cpp
WebCore/ksvg2/svg/SVGAnimationElement.h

index b216627e5a2f218056afe974711af40677cba103..a600ced738cd8e7bb18e485a3a555053e0f898d4 100644 (file)
@@ -1,3 +1,33 @@
+2006-10-09  Rob Buis  <buis@kde.org>
+
+        Reviewed by Mitz.
+
+        http://bugs.webkit.org/show_bug.cgi?id=11217
+        Cleanup svg coding style
+
+        Cleanup coding style in svg animation classes.
+
+        * ksvg2/svg/SVGAnimateColorElement.cpp:
+        (WebCore::SVGAnimateColorElement::handleTimerEvent):
+        * ksvg2/svg/SVGAnimateElement.cpp:
+        (WebCore::SVGAnimateElement::SVGAnimateElement):
+        (WebCore::SVGAnimateElement::handleTimerEvent):
+        * ksvg2/svg/SVGAnimateTransformElement.cpp:
+        (WebCore::SVGAnimateTransformElement::SVGAnimateTransformElement):
+        (WebCore::SVGAnimateTransformElement::parseMappedAttribute):
+        (WebCore::SVGAnimateTransformElement::handleTimerEvent):
+        (WebCore::SVGAnimateTransformElement::parseTransformValue):
+        (WebCore::SVGAnimateTransformElement::calculateRotationFromMatrix):
+        (WebCore::SVGAnimateTransformElement::initialMatrix):
+        (WebCore::SVGAnimateTransformElement::transformMatrix):
+        * ksvg2/svg/SVGAnimateTransformElement.h:
+        * ksvg2/svg/SVGAnimatedPathData.cpp:
+        * ksvg2/svg/SVGAnimatedPathData.h:
+        * ksvg2/svg/SVGAnimatedPoints.cpp:
+        * ksvg2/svg/SVGAnimatedPoints.h:
+        * ksvg2/svg/SVGAnimationElement.cpp:
+        (WebCore::SVGAnimationElement::targetElement):
+
 2006-10-09  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by Oliver.
index faa193c746d62080ad73f02e377a04811072382d..26d19db0f18f023a49d11858fe680891a328aeb7 100644 (file)
@@ -1,6 +1,6 @@
 /*
     Copyright (C) 2004, 2005 Nikolas Zimmermann <wildfox@kde.org>
-                  2004, 2005 Rob Buis <buis@kde.org>
+                  2004, 2005, 2006 Rob Buis <buis@kde.org>
 
     This file is part of the KDE project
 
 
 #include "config.h"
 #ifdef SVG_SUPPORT
-
-#include "SVGColor.h"
 #include "SVGAnimateColorElement.h"
+
+#include "Document.h"
 #include "KSVGTimeScheduler.h"
 #include "PlatformString.h"
-#include "Document.h"
-#include "SVGDocumentExtensions.h"
+#include "SVGColor.h"
 #include "SVGSVGElement.h"
 #include <math.h>
 
 namespace WebCore {
 
-SVGAnimateColorElement::SVGAnimateColorElement(const QualifiedName& tagName, Document *doc)
+SVGAnimateColorElement::SVGAnimateColorElement(const QualifiedName& tagName, Documentdoc)
     : SVGAnimationElement(tagName, doc)
     , m_toColor(new SVGColor())
     , m_fromColor(new SVGColor())
@@ -60,8 +59,7 @@ void SVGAnimateColorElement::handleTimerEvent(double timePercentage)
         m_initialColor = temp->color();
 
         // Animation mode handling
-        switch(detectAnimationMode())
-        {
+        switch (detectAnimationMode()) {
             case TO_ANIMATION:
             case FROM_TO_ANIMATION:
             {
index b964b3aaa41bfbb99671a87e12567539895c761e..c710e569de2b283bdf7f097b8e227a43fbdbbf82 100644 (file)
@@ -1,6 +1,6 @@
 /*
     Copyright (C) 2004, 2005 Nikolas Zimmermann <wildfox@kde.org>
-                  2004, 2005 Rob Buis <buis@kde.org>
+                  2004, 2005, 2006 Rob Buis <buis@kde.org>
 
     This file is part of the KDE project
 
 #ifdef SVG_SUPPORT
 
 #include "SVGAnimationElement.h"
-
-class SVGColor;
+#include <wtf/RefPtr.h>
 
 namespace WebCore {
 
+    class SVGColor;
+
     class SVGAnimateColorElement : public SVGAnimationElement {
     public:
         SVGAnimateColorElement(const QualifiedName&, Document*);
index d81fdab44737287c481df4bd22a07229158aa906..a688b89967851aec12cfc1ffadb886f75b987d3f 100644 (file)
@@ -1,6 +1,6 @@
 /*
     Copyright (C) 2004, 2005 Nikolas Zimmermann <wildfox@kde.org>
-                  2004, 2005 Rob Buis <buis@kde.org>
+                  2004, 2005, 2006 Rob Buis <buis@kde.org>
 
     This file is part of the KDE project
 
 #include "SVGAnimateElement.h"
 
 #include "KSVGTimeScheduler.h"
-#include "Document.h"
 #include "SVGDocumentExtensions.h"
 #include "SVGSVGElement.h"
 
-using namespace WebCore;
+namespace WebCore {
 
-SVGAnimateElement::SVGAnimateElement(const QualifiedName& tagName, Document *doc)
+SVGAnimateElement::SVGAnimateElement(const QualifiedName& tagName, Documentdoc)
     : SVGAnimationElement(tagName, doc)
     , m_currentItem(-1)
 {
@@ -50,8 +49,7 @@ void SVGAnimateElement::handleTimerEvent(double timePercentage)
         m_savedTo = attr.deprecatedString();
 
         // Animation mode handling
-        switch(detectAnimationMode())
-        {
+        switch (detectAnimationMode()) {
 /*            case TO_ANIMATION:
             case FROM_TO_ANIMATION:
             {
@@ -207,6 +205,8 @@ void SVGAnimateElement::handleTimerEvent(double timePercentage)
     }
 }
 
+}
+
 // vim:ts=4:noet
 #endif // SVG_SUPPORT
 
index bfe03f5e9c495e848666c1e49d262d159a5f7be0..ca9bd60c1caf7b35c57f82ad99a74d2c524a4a09 100644 (file)
@@ -1,6 +1,6 @@
 /*
     Copyright (C) 2004, 2005 Nikolas Zimmermann <wildfox@kde.org>
-                  2004, 2005 Rob Buis <buis@kde.org>
+                  2004, 2005, 2006 Rob Buis <buis@kde.org>
 
     This file is part of the KDE project
 
 
 #include "config.h"
 #ifdef SVG_SUPPORT
-#include "Attr.h"
-
-#include <kcanvas/RenderPath.h>
+#include "SVGAnimateTransformElement.h"
 
+#include "KSVGTimeScheduler.h"
 #include "SVGAngle.h"
 #include "SVGMatrix.h"
-#include "SVGTransform.h"
 #include "SVGStyledTransformableElement.h"
-#include "SVGTransformList.h"
-#include "SVGTransformList.h"
-#include "SVGAnimateTransformElement.h"
-#include "KSVGTimeScheduler.h"
-#include "Document.h"
-#include "SVGDocumentExtensions.h"
 #include "SVGSVGElement.h"
-
+#include "SVGTransform.h"
+#include "SVGTransformList.h"
 #include <math.h>
 
 using namespace std;
 
 namespace WebCore {
 
-SVGAnimateTransformElement::SVGAnimateTransformElement(const QualifiedName& tagName, Document *doc)
+SVGAnimateTransformElement::SVGAnimateTransformElement(const QualifiedName& tagName, Documentdoc)
     : SVGAnimationElement(tagName, doc)
     , m_currentItem(-1)
     , m_type(SVGTransform::SVG_TRANSFORM_UNKNOWN)
@@ -58,7 +51,7 @@ SVGAnimateTransformElement::~SVGAnimateTransformElement()
 {
 }
 
-void SVGAnimateTransformElement::parseMappedAttribute(MappedAttribute *attr)
+void SVGAnimateTransformElement::parseMappedAttribute(MappedAttributeattr)
 {
     if (attr->name() == SVGNames::typeAttr) {
         const String& value = attr->value();
@@ -84,12 +77,12 @@ void SVGAnimateTransformElement::handleTimerEvent(double timePercentage)
         
         // Save initial transform... (needed for fill="remove" or additve="sum")
         if (targetElement()->isStyledTransformable()) {
-            SVGStyledTransformableElement *transform = static_cast<SVGStyledTransformableElement *>(targetElement());
+            SVGStyledTransformableElement* transform = static_cast<SVGStyledTransformableElement*>(targetElement());
             RefPtr<SVGTransformList> transformList = transform->transformBaseValue();
             if (transformList) {
                 ExceptionCode ec = 0;
                 for (unsigned long i = 0; i < transformList->numberOfItems(); i++) {
-                    SVGTransform *value = transformList->getItem(i, ec).get();;
+                    SVGTransformvalue = transformList->getItem(i, ec).get();;
                     if (!value)
                         continue;
                         
@@ -102,8 +95,7 @@ void SVGAnimateTransformElement::handleTimerEvent(double timePercentage)
         }
                 
         // Animation mode handling
-        switch(detectAnimationMode())
-        {
+        switch (detectAnimationMode()) {
             case TO_ANIMATION:
             case FROM_TO_ANIMATION:
             {        
@@ -146,8 +138,8 @@ void SVGAnimateTransformElement::handleTimerEvent(double timePercentage)
                 if (!m_fromTransform)
                     m_fromTransform = new SVGTransform();
 
-                SVGMatrix *byMatrix = m_toTransform->matrix();
-                SVGMatrix *fromMatrix = m_fromTransform->matrix();
+                SVGMatrixbyMatrix = m_toTransform->matrix();
+                SVGMatrixfromMatrix = m_fromTransform->matrix();
 
                 byMatrix->multiply(fromMatrix);
 
@@ -231,8 +223,7 @@ void SVGAnimateTransformElement::handleTimerEvent(double timePercentage)
             m_transformMatrix->multiply(m_lastMatrix.get());
     }
     
-    switch(m_type)
-    {
+    switch (m_type) {
         case SVGTransform::SVG_TRANSFORM_TRANSLATE:
         {
             double dx = ((qToMatrix.dx() - qFromMatrix.dx()) * useTimePercentage) + qFromMatrix.dx();
@@ -322,7 +313,7 @@ void SVGAnimateTransformElement::handleTimerEvent(double timePercentage)
         m_initialTransform = 0;
 
         if (!isFrozen()) {
-            SVGMatrix *initial = initialMatrix();
+            SVGMatrixinitial = initialMatrix();
             if (initial)
                 m_transformMatrix = initial;
             else
@@ -341,8 +332,7 @@ RefPtr<SVGTransform> SVGAnimateTransformElement::parseTransformValue(const Strin
 
     RefPtr<SVGTransform> parsedTransform = new SVGTransform();
     
-    switch(m_type)
-    {
+    switch (m_type) {
         case SVGTransform::SVG_TRANSFORM_TRANSLATE:
         {
             double tx = 0.0, ty = 0.0;
@@ -421,7 +411,7 @@ RefPtr<SVGTransform> SVGAnimateTransformElement::parseTransformValue(const Strin
     return parsedTransform;
 }
 
-void SVGAnimateTransformElement::calculateRotationFromMatrix(const AffineTransform &matrix, double &angle, double &cx, double &cy) const
+void SVGAnimateTransformElement::calculateRotationFromMatrix(const AffineTransform& matrix, double& angle, double& cx, double& cy) const
 {
     double cosa = matrix.m11();
     double sina = -matrix.m21();
@@ -447,12 +437,12 @@ void SVGAnimateTransformElement::calculateRotationFromMatrix(const AffineTransfo
     angle = 0.0;
 }
 
-SVGMatrix *SVGAnimateTransformElement::initialMatrix() const
+SVGMatrixSVGAnimateTransformElement::initialMatrix() const
 {
     if (!targetElement()->isStyledTransformable())
         return 0;
-    SVGStyledTransformableElement *transform = static_cast<SVGStyledTransformableElement *>(targetElement());
-    SVGTransformList *transformList = (transform ? transform->transformBaseValue() : 0);
+    SVGStyledTransformableElement* transform = static_cast<SVGStyledTransformableElement*>(targetElement());
+    SVGTransformListtransformList = (transform ? transform->transformBaseValue() : 0);
     if (!transformList)
         return 0;
     
@@ -460,13 +450,13 @@ SVGMatrix *SVGAnimateTransformElement::initialMatrix() const
     if (!result)
         return 0;
 
-    SVGMatrix *ret = result->matrix();
+    SVGMatrixret = result->matrix();
     ret->ref();
 
     return ret;
 }
 
-SVGMatrix *SVGAnimateTransformElement::transformMatrix() const
+SVGMatrixSVGAnimateTransformElement::transformMatrix() const
 {
     return m_transformMatrix.get();
 }
index 01538fce195ae71b8227aa7f08e60ddffa6acc58..0eb1b728e418f024bd62b245caa95cb94f1036d0 100644 (file)
@@ -1,6 +1,6 @@
 /*
     Copyright (C) 2004, 2005 Nikolas Zimmermann <wildfox@kde.org>
-                  2004, 2005 Rob Buis <buis@kde.org>
+                  2004, 2005, 2006 Rob Buis <buis@kde.org>
 
     This file is part of the KDE project
 
@@ -24,8 +24,8 @@
 #define SVGAnimateTransformElement_H
 #ifdef SVG_SUPPORT
 
-#include "SVGTransform.h"
 #include "SVGAnimationElement.h"
+#include "SVGTransform.h"
 
 namespace WebCore {
 
@@ -42,7 +42,7 @@ namespace WebCore {
 
         // Helpers
         RefPtr<SVGTransform> parseTransformValue(const String&) const;
-        void calculateRotationFromMatrix(const AffineTransform&, double &angle, double &cx, double &cy) const;
+        void calculateRotationFromMatrix(const AffineTransform&, double& angle, double& cx, double& cy) const;
 
         SVGMatrix* initialMatrix() const;
         SVGMatrix* transformMatrix() const;
index e4a0c5b75b39248be65e201799c44f9de84fed82..e01e0d013e6ba9d69ff6f455df39f08e432d3bfd 100644 (file)
@@ -1,6 +1,6 @@
 /*
     Copyright (C) 2004, 2005 Nikolas Zimmermann <wildfox@kde.org>
-                  2004, 2005 Rob Buis <buis@kde.org>
+                  2004, 2005, 2006 Rob Buis <buis@kde.org>
 
     This file is part of the KDE project
 
@@ -24,7 +24,7 @@
 #ifdef SVG_SUPPORT
 #include "SVGAnimatedPathData.h"
 
-using namespace WebCore;
+namespace WebCore {
 
 SVGAnimatedPathData::SVGAnimatedPathData()
 {
@@ -34,6 +34,8 @@ SVGAnimatedPathData::~SVGAnimatedPathData()
 {
 }
 
+}
+
 // vim:ts=4:noet
 #endif // SVG_SUPPORT
 
index 408fb27715e909b168e195e8d5ee0c43d32a6c29..1c5954d613d870b3b3a73912f56129dd1c51bed4 100644 (file)
@@ -1,6 +1,6 @@
 /*
     Copyright (C) 2004, 2005 Nikolas Zimmermann <wildfox@kde.org>
-                  2004, 2005 Rob Buis <buis@kde.org>
+                  2004, 2005, 2006 Rob Buis <buis@kde.org>
 
     This file is part of the KDE project
 
@@ -34,10 +34,10 @@ namespace WebCore
         virtual ~SVGAnimatedPathData();
 
         // 'SVGAnimatedPathData' functions
-        virtual SVGPathSegList *pathSegList() const = 0;
-        virtual SVGPathSegList *normalizedPathSegList() const = 0;
-        virtual SVGPathSegList *animatedPathSegList() const = 0;
-        virtual SVGPathSegList *animatedNormalizedPathSegList() const = 0;
+        virtual SVGPathSegListpathSegList() const = 0;
+        virtual SVGPathSegListnormalizedPathSegList() const = 0;
+        virtual SVGPathSegListanimatedPathSegList() const = 0;
+        virtual SVGPathSegListanimatedNormalizedPathSegList() const = 0;
     };
 
 } // namespace WebCore
index 782558d51ff76c5b4f6555c9e3b43b14f8901928..685daa9cecc6938f82f9fdec368babebf852dae2 100644 (file)
@@ -1,6 +1,6 @@
 /*
     Copyright (C) 2004, 2005 Nikolas Zimmermann <wildfox@kde.org>
-                  2004, 2005 Rob Buis <buis@kde.org>
+                  2004, 2005, 2006 Rob Buis <buis@kde.org>
 
     This file is part of the KDE project
 
@@ -24,7 +24,7 @@
 #ifdef SVG_SUPPORT
 #include "SVGAnimatedPoints.h"
 
-using namespace WebCore;
+namespace WebCore {
 
 SVGAnimatedPoints::SVGAnimatedPoints()
 {
@@ -34,6 +34,8 @@ SVGAnimatedPoints::~SVGAnimatedPoints()
 {
 }
 
+}
+
 // vim:ts=4:noet
 #endif // SVG_SUPPORT
 
index 77f1ee1d0c1752921e36c352173176409f3d7105..e4a9b0c423bc535adae197b9c9d621beaa6f0798 100644 (file)
@@ -1,6 +1,6 @@
 /*
     Copyright (C) 2004, 2005 Nikolas Zimmermann <wildfox@kde.org>
-                  2004, 2005 Rob Buis <buis@kde.org>
+                  2004, 2005, 2006 Rob Buis <buis@kde.org>
 
     This file is part of the KDE project
 
@@ -34,8 +34,8 @@ namespace WebCore
         virtual ~SVGAnimatedPoints();
 
         // 'SVGAnimatedPoints' functions
-        virtual SVGPointList *points() const = 0;
-        virtual SVGPointList *animatedPoints() const = 0;
+        virtual SVGPointListpoints() const = 0;
+        virtual SVGPointListanimatedPoints() const = 0;
     };
 
 } // namespace WebCore
index ccbcb0d08b582293d05dd34d43b7631c0aff2923..11e7c4acc558f3fd305c191930f97d8ee75ad5ab 100644 (file)
@@ -1,6 +1,6 @@
 /*
     Copyright (C) 2004, 2005 Nikolas Zimmermann <wildfox@kde.org>
-                  2004, 2005 Rob Buis <buis@kde.org>
+                  2004, 2005, 2006 Rob Buis <buis@kde.org>
 
     This file is part of the KDE project
 
 #ifdef SVG_SUPPORT
 #include "SVGAnimationElement.h"
 
-#include "Attr.h"
 #include "CSSPropertyNames.h"
 #include "Document.h"
-#include "DOMImplementation.h"
-#include "ksvgcssproperties.h"
 #include "KSVGTimeScheduler.h"
 #include "PlatformString.h"
-#include "SVGDocumentExtensions.h"
 #include "SVGHelper.h"
-#include "SVGStyledElement.h"
 #include "SVGSVGElement.h"
+#include "SVGStyledElement.h"
 #include "SVGURIReference.h"
 #include "XLinkNames.h"
+#include "ksvgcssproperties.h"
 #include <float.h>
 #include <math.h>
 #include <wtf/Vector.h>
@@ -73,15 +70,15 @@ SVGAnimationElement::~SVGAnimationElement()
 {
 }
 
-SVGElement *SVGAnimationElement::targetElement() const
+SVGElementSVGAnimationElement::targetElement() const
 {
     if (!m_targetElement) {
         if (!m_href.isEmpty()) {
-            Element *element = ownerDocument()->getElementById(SVGURIReference::getTarget(m_href));
+            Elementelement = ownerDocument()->getElementById(SVGURIReference::getTarget(m_href));
             m_targetElement = svg_dynamic_cast(element);
         } else if (parentNode()) {
-            Node *target = parentNode();
-            while(target != 0) {
+            Nodetarget = parentNode();
+            while (target) {
                 if (target->nodeType() != ELEMENT_NODE)
                     target = target->parentNode();
                 else
@@ -114,7 +111,7 @@ double SVGAnimationElement::getSimpleDuration(ExceptionCode&) const
     return m_simpleDuration;
 }
 
-void SVGAnimationElement::parseMappedAttribute(MappedAttribute *attr)
+void SVGAnimationElement::parseMappedAttribute(MappedAttributeattr)
 {
     const String& value = attr->value();
     if (attr->name().matches(XLinkNames::hrefAttr))
@@ -400,10 +397,10 @@ String SVGAnimationElement::targetAttribute() const
     if (!targetElement())
         return String();
     
-    SVGElement *target = targetElement();
-    SVGStyledElement *styled = NULL;
+    SVGElementtarget = targetElement();
+    SVGStyledElement* styled = 0;
     if (target && target->isStyled())
-        styled = static_cast<SVGStyledElement *>(target);
+        styled = static_cast<SVGStyledElement*>(target);
     
     String ret;
 
index 63e510370051b075a5b0a3a12f3b9d5ab4d8e555..8ad6cfbacbc2c9aec654212122cc11c2601e0c66 100644 (file)
@@ -1,6 +1,6 @@
 /*
     Copyright (C) 2004, 2005 Nikolas Zimmermann <wildfox@kde.org>
-                  2004, 2005 Rob Buis <buis@kde.org>
+                  2004, 2005, 2006 Rob Buis <buis@kde.org>
 
     This file is part of the KDE project
 
@@ -75,8 +75,8 @@ namespace WebCore {
     };
 
     class SVGAnimationElement : public SVGElement,
-                                    public SVGTests,
-                                    public SVGExternalResourcesRequired
+                                public SVGTests,
+                                public SVGExternalResourcesRequired
     {
     public:
         SVGAnimationElement(const QualifiedName&, Document*);
@@ -92,7 +92,7 @@ namespace WebCore {
         double getCurrentTime() const;
         double getSimpleDuration(ExceptionCode&) const;
     
-        virtual void parseMappedAttribute(MappedAttribute *attr);
+        virtual void parseMappedAttribute(MappedAttributeattr);
         virtual bool rendererIsNeeded(RenderStyle*) { return false; }
 
         virtual void closeRenderer();