Title: [247641] trunk/Source/WebCore
Revision
247641
Author
commit-qu...@webkit.org
Date
2019-07-19 02:11:35 -0700 (Fri, 19 Jul 2019)

Log Message

Remove some unneeded code in MathOperator
https://bugs.webkit.org/show_bug.cgi?id=199935

Patch by Rob Buis <rb...@igalia.com> on 2019-07-19
Reviewed by Frédéric Wang.

The initialize method is unneeded since the member
variables all use default initialization.

No new tests, not changing behavior.

* rendering/mathml/MathOperator.cpp:
(WebCore::MathOperator::MathOperator):
(WebCore::MathOperator::GlyphAssemblyData::initialize): Deleted.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (247640 => 247641)


--- trunk/Source/WebCore/ChangeLog	2019-07-19 08:34:57 UTC (rev 247640)
+++ trunk/Source/WebCore/ChangeLog	2019-07-19 09:11:35 UTC (rev 247641)
@@ -1,3 +1,19 @@
+2019-07-19  Rob Buis  <rb...@igalia.com>
+
+        Remove some unneeded code in MathOperator
+        https://bugs.webkit.org/show_bug.cgi?id=199935
+
+        Reviewed by Frédéric Wang.
+
+        The initialize method is unneeded since the member
+        variables all use default initialization.
+
+        No new tests, not changing behavior.
+
+        * rendering/mathml/MathOperator.cpp:
+        (WebCore::MathOperator::MathOperator):
+        (WebCore::MathOperator::GlyphAssemblyData::initialize): Deleted.
+
 2019-07-18  Antoine Quint  <grao...@apple.com>
 
         Enable simulated mouse events quirk for soundcloud.com

Modified: trunk/Source/WebCore/rendering/mathml/MathOperator.cpp (247640 => 247641)


--- trunk/Source/WebCore/rendering/mathml/MathOperator.cpp	2019-07-19 08:34:57 UTC (rev 247640)
+++ trunk/Source/WebCore/rendering/mathml/MathOperator.cpp	2019-07-19 09:11:35 UTC (rev 247641)
@@ -86,21 +86,8 @@
     { 0x222b, 0x2320, 0x23ae, 0x2321, 0x0    } // integral sign
 };
 
-void MathOperator::GlyphAssemblyData::initialize()
-{
-    topOrRightCodePoint = 0;
-    topOrRightFallbackGlyph = 0;
-    extensionCodePoint = 0;
-    extensionFallbackGlyph = 0;
-    bottomOrLeftCodePoint = 0;
-    bottomOrLeftFallbackGlyph = 0;
-    middleCodePoint = 0;
-    middleFallbackGlyph = 0;
-}
-    
 MathOperator::MathOperator()
 {
-    m_assembly.initialize();
     m_variantGlyph = 0;
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to