Bug 22468 – Coding style: config.h should only be included in implementation files, not in header

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Julien Chaffraix]

See https://bugs.webkit.org/show_bug.cgi?id=22441#c7 for Darin's explanation on that subject.

[Julien Chaffraix]

Created attachment 25509 [details]
Proposed addition: mostly made for discussion & enhancement

[Eric Seidel (no email)]

Comment on attachment 25509 [details]
Proposed addition: mostly made for discussion & enhancement

I'm not sure it's clear to me from reading what an "implementation" file is.  Maybe (.cpp, .c, .mm, etc.) as an aside would help.

This sentence is backwards:
This guarantees that each header's completeness is tested,
+to make sure it can be compiled without requiring any other header files be
+included first.

"This assures that each header can be compiled without requiring any other header included first."  or similar, no need to state "completeness" and then define it.

An extra wrong example of HTMLDivElement.h might be helpful.

Having HTMLDivElement.cpp unsorted might also be useful. :)

Otherwise looks fine.  This is a useful change to the style guide.

[Julien Chaffraix]

Created attachment 25540 [details]
Updated proposition: add one more example, explains a bit what an implementation file is

[Eric Seidel (no email)]

Comment on attachment 25540 [details]
Updated proposition: add one more example, explains a bit what an implementation file is

Looks good.  Thanks!

[Julien Chaffraix]

Landed in r38824.