Title: [108540] trunk/Tools
Revision
108540
Author
commit-qu...@webkit.org
Date
2012-02-22 12:51:32 -0800 (Wed, 22 Feb 2012)

Log Message

Explicitly include unistd.h in TestNetscapePlugIn for GCC 4.7
https://bugs.webkit.org/show_bug.cgi?id=77759

Patch by Kalev Lember <kalevlem...@gmail.com> on 2012-02-22
Reviewed by Tony Chang.

Fixes build with gcc 4.7.

r100432 added the include only for ANDROID; this changes the ifdef to
also cover other unix platforms. On Fedora 17 the gcc 4.7 compiler no
longer implicitly includes unistd.h from standard headers and other
unix platforms are likely to get gcc 4.7 as well in the future.

* DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (108539 => 108540)


--- trunk/Tools/ChangeLog	2012-02-22 20:43:22 UTC (rev 108539)
+++ trunk/Tools/ChangeLog	2012-02-22 20:51:32 UTC (rev 108540)
@@ -1,3 +1,19 @@
+2012-02-22  Kalev Lember  <kalevlem...@gmail.com>
+
+        Explicitly include unistd.h in TestNetscapePlugIn for GCC 4.7
+        https://bugs.webkit.org/show_bug.cgi?id=77759
+
+        Reviewed by Tony Chang.
+
+        Fixes build with gcc 4.7.
+
+        r100432 added the include only for ANDROID; this changes the ifdef to
+        also cover other unix platforms. On Fedora 17 the gcc 4.7 compiler no
+        longer implicitly includes unistd.h from standard headers and other
+        unix platforms are likely to get gcc 4.7 as well in the future.
+
+        * DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp:
+
 2012-02-22  Philippe Normand  <pnorm...@igalia.com>
 
         Unreviewed, rolling out r108522.

Modified: trunk/Tools/DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp (108539 => 108540)


--- trunk/Tools/DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp	2012-02-22 20:43:22 UTC (rev 108539)
+++ trunk/Tools/DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp	2012-02-22 20:51:32 UTC (rev 108540)
@@ -29,7 +29,7 @@
 #include <assert.h>
 #include <string.h>
 
-#if defined(ANDROID)
+#if defined(XP_UNIX) || defined(ANDROID)
 #include <unistd.h>
 #endif
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to