Bug 619498 - Part 1: Eliminate some forwarding member functions with using declarations. r=dholbert
authorCameron McCormack <cam@mcc.id.au>
Fri, 08 Apr 2011 10:17:32 +1200
changeset 67627 9e8156f46a4a312b09469948ddc1e33c932ebd08
parent 67626 558fff008e746e4011790ad8dd5c6ed10a46f61f
child 67628 3f55422367a6022d06337a65f77ff06db8dbb2e0
push id1
push userroot
push dateTue, 26 Apr 2011 22:38:44 +0000
treeherdermozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdholbert
bugs619498
milestone2.2a1pre
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 619498 - Part 1: Eliminate some forwarding member functions with using declarations. r=dholbert
content/svg/content/src/SVGPathData.h
--- a/content/svg/content/src/SVGPathData.h
+++ b/content/svg/content/src/SVGPathData.h
@@ -116,17 +116,17 @@ public:
   void GetValueAsString(nsAString& aValue) const;
 
   PRBool IsEmpty() const {
     return mData.IsEmpty();
   }
 
 #ifdef DEBUG
   /**
-   * This method iterates over the encoded segment data and countes the number
+   * This method iterates over the encoded segment data and counts the number
    * of segments we currently have.
    */
   PRUint32 CountItems() const;
 #endif
 
   /**
    * Returns the number of *floats* in the encoding array, and NOT the number
    * of segments encoded in this object. (For that, see CountItems() above.)
@@ -254,28 +254,19 @@ public:
     return SVGPathData::CopyFrom(rhs);
   }
 
   /**
    * Exposed so that SVGPathData baseVals can be copied to
    * SVGPathDataAndOwner objects. Note that callers should also call
    * SetElement() when using this method!
    */
-  nsresult CopyFrom(const SVGPathData& rhs) {
-    return SVGPathData::CopyFrom(rhs);
-  }
-  const float& operator[](PRUint32 aIndex) const {
-    return SVGPathData::operator[](aIndex);
-  }
-  float& operator[](PRUint32 aIndex) {
-    return SVGPathData::operator[](aIndex);
-  }
-  PRBool SetLength(PRUint32 aNumberOfItems) {
-    return SVGPathData::SetLength(aNumberOfItems);
-  }
+  using SVGPathData::CopyFrom;
+  using SVGPathData::operator[];
+  using SVGPathData::SetLength;
 
 private:
   // We must keep a strong reference to our element because we may belong to a
   // cached baseVal nsSMILValue. See the comments starting at:
   // https://bugzilla.mozilla.org/show_bug.cgi?id=515116#c15
   nsRefPtr<nsSVGElement> mElement;
 };