Merge r185195 - Shrink the ArabicCharShapingMode enum in SVGGlyph.cpp

https://bugs.webkit.org/show_bug.cgi?id=145564 Reviewed by Darin Adler. Shrink the ArabicCharShapingMode enum to just one byte. This drops the size of the static s_arabicCharShapingMode array of ArabicCharShapingMode values from 888 bytes to 222. * platform/graphics/SVGGlyph.cpp: (WebCore::processArabicFormDetection):
WebKit · Jul 6, 2015 · 467ede2 · 467ede2
1 parent cbe91c9
commit 467ede2
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
@@ -1,3 +1,17 @@
+2015-06-04  Zan Dobersek  <zdobersek@igalia.com>
+
+        Shrink the ArabicCharShapingMode enum in SVGGlyph.cpp
+        https://bugs.webkit.org/show_bug.cgi?id=145564
+
+        Reviewed by Darin Adler.
+
+        Shrink the ArabicCharShapingMode enum to just one byte.
+        This drops the size of the static s_arabicCharShapingMode
+        array of  ArabicCharShapingMode values from 888 bytes to 222.
+
+        * platform/graphics/SVGGlyph.cpp:
+        (WebCore::processArabicFormDetection):
+
 2015-06-03  Zalan Bujtas  <zalan@apple.com>
 
         Subpixel rendering: Composited layer with subpixel gap does not get painted properly when its position changes.

diff --git a/Source/WebCore/platform/graphics/SVGGlyph.cpp b/Source/WebCore/platform/graphics/SVGGlyph.cpp
@@ -26,7 +26,7 @@
 namespace WebCore {
 
 // Helper functions to determine the arabic character forms (initial, medial, terminal, isolated)
-enum ArabicCharShapingMode {
+enum ArabicCharShapingMode : uint8_t {
     SNone = 0,
     SRight = 1,
     SDual = 2