[webkit-reviews] review requested: [Bug 77759] Missing include in TestNetscapePlugIn : [Attachment 125376] Explicitly include unistd.h in TestNetscapePlugIn

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Feb 3 11:44:14 PST 2012


Kalev Lember <kalevlember at gmail.com> has asked	for review:
Bug 77759: Missing include in TestNetscapePlugIn
https://bugs.webkit.org/show_bug.cgi?id=77759

Attachment 125376: Explicitly include unistd.h in TestNetscapePlugIn
https://bugs.webkit.org/attachment.cgi?id=125376&action=review

------- Additional Comments from Kalev Lember <kalevlember at gmail.com>
Fixes build with gcc 4.7.

r100432 added the include only for ANDROID; this changes the ifdef to
cover all 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.


More information about the webkit-reviews mailing list