[Webkit-unassigned] [Bug 105733] New: Let check-webkit-style check for Platfom.h inclusion
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Dec 24 18:57:40 PST 2012
https://bugs.webkit.org/show_bug.cgi?id=105733
Summary: Let check-webkit-style check for Platfom.h inclusion
Product: WebKit
Version: 528+ (Nightly build)
Platform: Unspecified
OS/Version: Unspecified
Status: NEW
Severity: Normal
Priority: P2
Component: Tools / Tests
AssignedTo: webkit-unassigned at lists.webkit.org
ReportedBy: laszlo.gombos at webkit.org
According to WebKit coding style - "All implementation files must #include "config.h" first. Header files should never include "config.h"
As config.h (by design) includes wtf/Platform.h including wtf/Platform.h explicitly in {c|cpp} files is always a duplicate/unnecessary.
We might consider having also some rule for .h files as well, but that needs a bit more discussion.
See also bug 105678.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list