Changeset 140473 in webkit

Timestamp:

Jan 22, 2013 2:46:35 PM (11 years ago)

Author:

hclam@chromium.org

Message:

Make CompactHTMLToken a little more compact
​https://bugs.webkit.org/show_bug.cgi?id=107317

Patch by Eric Seidel <​eric@webkit.org> on 2013-01-22
Reviewed by Adam Barth.

The windows builders got sad because MSVC won't combine
adjacent bit-fields with different types. Changing
bool to unsigned should fix the problem.

• html/parser/CompactHTMLToken.h:

(CompactHTMLToken):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/parser/CompactHTMLToken.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-01-22  Eric Seidel  <eric@webkit.org>
+
+        Make CompactHTMLToken a little more compact
+        https://bugs.webkit.org/show_bug.cgi?id=107317
+
+        Reviewed by Adam Barth.
+
+        The windows builders got sad because MSVC won't combine
+        adjacent bit-fields with different types.  Changing
+        bool to unsigned should fix the problem.
+
+        * html/parser/CompactHTMLToken.h:
+        (CompactHTMLToken):
+
 2013-01-22  Anthony Scian  <ascian@rim.com>
 

trunk/Source/WebCore/html/parser/CompactHTMLToken.h

@@ -75 +75 @@
 private:
     unsigned m_type : 4;
-    bool m_selfClosing : 1;
+    unsigned m_selfClosing : 1;
 
     String m_data; // "name", "characters", or "data" depending on m_type