Title: [110481] trunk/Source/WebCore
Revision
110481
Author
[email protected]
Date
2012-03-12 14:36:10 -0700 (Mon, 12 Mar 2012)

Log 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):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (110480 => 110481)


--- trunk/Source/WebCore/ChangeLog	2012-03-12 21:33:48 UTC (rev 110480)
+++ trunk/Source/WebCore/ChangeLog	2012-03-12 21:36:10 UTC (rev 110481)
@@ -1,3 +1,16 @@
+2012-03-12  George Staikos  <[email protected]>
+
+        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  <[email protected]>
 
         https://bugs.webkit.org/show_bug.cgi?id=80888

Modified: trunk/Source/WebCore/platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.h (110480 => 110481)


--- trunk/Source/WebCore/platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.h	2012-03-12 21:33:48 UTC (rev 110480)
+++ trunk/Source/WebCore/platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.h	2012-03-12 21:36:10 UTC (rev 110481)
@@ -83,7 +83,7 @@
     void glyphsForRange(int from, int to, int& fromGlyph, int& glyphLength);
 
     // 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
     // by the current x offset and that the x offset is updated for each script
@@ -91,10 +91,10 @@
     const SkPoint* positions() const { return m_positions; }
 
     // 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); }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to