SVGGlyph wastes space due to padding
authormmaxfield@apple.com <mmaxfield@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 1 Aug 2014 22:33:50 +0000 (22:33 +0000)
committermmaxfield@apple.com <mmaxfield@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 1 Aug 2014 22:33:50 +0000 (22:33 +0000)
https://bugs.webkit.org/show_bug.cgi?id=135520

Reviewed by Dean Jackson.

Rearrange members in SVGGlyph so there is no wasted padding space.

No new tests because there is no behavior change.

* platform/graphics/SVGGlyph.h:
(WebCore::SVGGlyph::SVGGlyph):

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

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/SVGGlyph.h

index 791a5883ba379e2d6f970b74eaa4e21492e0e1de..0e6205388cdb409f8f939e59736b611241884c7a 100644 (file)
@@ -1,3 +1,17 @@
+2014-08-01  Myles C. Maxfield  <mmaxfield@apple.com>
+
+        SVGGlyph wastes space due to padding
+        https://bugs.webkit.org/show_bug.cgi?id=135520
+
+        Reviewed by Dean Jackson.
+
+        Rearrange members in SVGGlyph so there is no wasted padding space.
+
+        No new tests because there is no behavior change.
+
+        * platform/graphics/SVGGlyph.h:
+        (WebCore::SVGGlyph::SVGGlyph):
+
 2014-07-31  Dean Jackson  <dino@apple.com>
 
         [Media iOS] Ensure there is a nice default fallback for missing wireless target names
index 272811bb6be64fe5e4ca6813a3afda7470c16cb6..7f5f7f989b6dc111f929857c5acb71ba6c03c0ab 100644 (file)
@@ -51,16 +51,16 @@ struct SVGGlyph {
     };
 
     SVGGlyph()
-        : isPartOfLigature(false)
-        , orientation(Both)
-        , arabicForm(None)
+        : unicodeStringLength(0)
         , priority(0)
-        , tableEntry(0)
-        , unicodeStringLength(0)
         , horizontalAdvanceX(0)
         , verticalOriginX(0)
         , verticalOriginY(0)
         , verticalAdvanceY(0)
+        , tableEntry(0)
+        , isPartOfLigature(false)
+        , orientation(Both)
+        , arabicForm(None)
     {
     }
 
@@ -86,22 +86,25 @@ struct SVGGlyph {
             && languages == other.languages;
     }
 
-    bool isPartOfLigature : 1;
+    Vector<String> languages;
 
-    unsigned orientation : 2; // Orientation
-    unsigned arabicForm : 3; // ArabicForm
-    int priority;
-    Glyph tableEntry;
     size_t unicodeStringLength;
     String glyphName;
 
+    Path pathData;
+    int priority;
+
     float horizontalAdvanceX;
     float verticalOriginX;
     float verticalOriginY;
     float verticalAdvanceY;
 
-    Path pathData;
-    Vector<String> languages;
+    Glyph tableEntry;
+
+    unsigned isPartOfLigature : 1;
+
+    unsigned orientation : 2; // Orientation
+    unsigned arabicForm : 3; // ArabicForm
 };
 
 Vector<SVGGlyph::ArabicForm> charactersWithArabicForm(const String& input, bool rtl);