Changeset 110481 in webkit

Timestamp:

Mar 12, 2012 2:36:10 PM (12 years ago)

Author:

staikos@webkit.org

Message:

It doesn't make sense to return const unsigned, and GCC warns about it.
Remove const.
​https://bugs.webkit.org/show_bug.cgi?id=80790

Reviewed by Antonio Gomes.

• platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.h:

(WebCore::ComplexTextController::length):
(WebCore::ComplexTextController::numCodePoints):
(WebCore::ComplexTextController::offsetX):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.h (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-03-12  George Staikos  <staikos@webkit.org>
+
+        It doesn't make sense to return const unsigned, and GCC warns about it.
+        Remove const.
+        https://bugs.webkit.org/show_bug.cgi?id=80790
+
+        Reviewed by Antonio Gomes.
+
+        * platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.h:
+        (WebCore::ComplexTextController::length):
+        (WebCore::ComplexTextController::numCodePoints):
+        (WebCore::ComplexTextController::offsetX):
+
 2012-03-12  Beth Dakin  <bdakin@apple.com>
 

trunk/Source/WebCore/platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.h

@@ -84 +84 @@
 
     // Return the length of the array returned by |glyphs|
-    const unsigned length() const { return m_item.num_glyphs; }
+    unsigned length() const { return m_item.num_glyphs; }
 
     // Return the offset for each of the glyphs. Note that this is translated
@@ -92 +92 @@
 
     // return the number of code points in the current script run
-    const unsigned numCodePoints() const { return m_item.item.length; }
+    unsigned numCodePoints() const { return m_item.item.length; }
 
     // Return the current pixel position of the controller.
-    const unsigned offsetX() const { return m_offsetX; }
+    unsigned offsetX() const { return m_offsetX; }
 
     const FontPlatformData* fontPlatformDataForScriptRun() { return reinterpret_cast<FontPlatformData*>(m_item.font->userData); }