Changeset 247141 in webkit

Timestamp:

Jul 4, 2019 6:54:50 AM (5 years ago)

Author:

Konstantin Tokarev

Message:

HyphenationLibHyphen: Include GLib-related headers only for PLATFORM(GTK)
​https://bugs.webkit.org/show_bug.cgi?id=199469

Reviewed by Michael Catanzaro.

Also added compile guard for a couple of functions used only in GTK port.

• platform/text/hyphen/HyphenationLibHyphen.cpp:

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/text/hyphen/HyphenationLibHyphen.cpp (modified) (3 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2019-07-04  Konstantin Tokarev  <annulen@yandex.ru>
+
+        HyphenationLibHyphen: Include GLib-related headers only for PLATFORM(GTK)
+        https://bugs.webkit.org/show_bug.cgi?id=199469
+
+        Reviewed by Michael Catanzaro.
+
+        Also added compile guard for a couple of functions used only in GTK port.
+
+        * platform/text/hyphen/HyphenationLibHyphen.cpp:
+
 2019-07-03  Eric Carlson  <eric.carlson@apple.com>
 

trunk/Source/WebCore/platform/text/hyphen/HyphenationLibHyphen.cpp

@@ -37 +37 @@
 #include <wtf/NeverDestroyed.h>
 #include <wtf/TinyLRUCache.h>
-#include <wtf/glib/GLibUtilities.h>
-#include <wtf/glib/GUniquePtr.h>
 #include <wtf/text/AtomStringHash.h>
 #include <wtf/text/CString.h>
 #include <wtf/text/StringView.h>
+
+#if PLATFORM(GTK)
+#include <wtf/glib/GLibUtilities.h>
+#include <wtf/glib/GUniquePtr.h>
+#endif
 
 namespace WebCore {
@@ -86 +89 @@
 
 #if ENABLE(DEVELOPER_MODE)
+
+#if PLATFORM(GTK)
 static CString topLevelPath()
 {
@@ -109 +114 @@
     return outputDir.get();
 }
+#endif // PLATFORM(GTK)
 
 static void scanTestDictionariesDirectoryIfNecessary(HashMap<AtomString, Vector<String>>& availableLocales)