Bug 41777 - Remove needless #include <fcntl.h> from TCSystemAlloc.cpp
Summary: Remove needless #include <fcntl.h> from TCSystemAlloc.cpp
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Template Framework (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-07 11:00 PDT by Patrick R. Gansterer
Modified: 2010-07-08 00:24 PDT (History)
1 user (show)

See Also:


Attachments
Patch (942 bytes, patch)
2010-07-07 11:01 PDT, Patrick R. Gansterer
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick R. Gansterer 2010-07-07 11:00:49 PDT
see patch
Comment 1 Patrick R. Gansterer 2010-07-07 11:01:54 PDT
Created attachment 60755 [details]
Patch
Comment 2 Darin Adler 2010-07-07 12:28:40 PDT
Comment on attachment 60755 [details]
Patch

It seems that that fcntl.h was needed for O_RDWR, which is used in TryDevMem, but that's no longer compiled in our copy of this file. So it would be slightly better to put <fcntl.h> inside #ifndef WTF_CHANGES, or maybe we should get rid of all the #ifndef WTF_CHANGES entirely.
Comment 3 WebKit Commit Bot 2010-07-08 00:24:11 PDT
Comment on attachment 60755 [details]
Patch

Clearing flags on attachment: 60755

Committed r62764: <http://trac.webkit.org/changeset/62764>
Comment 4 WebKit Commit Bot 2010-07-08 00:24:15 PDT
All reviewed patches have been landed.  Closing bug.