Bug 27382

Summary:

Build break -- including a corefoundation header

Product:

WebKit

Reporter:

Eric Roman <eroman>

Component:

WebCore Misc.

Assignee:

Eric Roman <eroman>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Attachments:

Description	Flags
Add a header inclusion guard	darin: review-
Add a header inclusion guard (v2)	darin: review+

Description Eric Roman 2009-07-17 11:44:20 PDT

http://trac.webkit.org/changeset/46042 add an include of "RetainPtr.h" in CollatorICU.cpp.

RetainPtr.h in turn includes:

include <CoreFoundation/CoreFoundation.h>

This breaks compile on non-mac builds for users of CollatorICU.cpp
(Broke chromium builder)

Comment 1 Eric Roman 2009-07-17 12:05:05 PDT

Created attachment 32962 [details]
Add a header inclusion guard

Comment 2 Darin Adler 2009-07-17 12:09:21 PDT

Comment on attachment 32962 [details]
Add a header inclusion guard

Conditionalized includes go in separate paragraphs in our coding style.

Typo "Build fox" instead of build fix. Would you be willing to do a new patch with those fixed?

Comment 3 Eric Roman 2009-07-17 12:18:07 PDT

Created attachment 32964 [details]
Add a header inclusion guard (v2)

New patch -- addressed darin@apple.com's feedback.

Comment 4 Eric Roman 2009-07-17 12:36:53 PDT

Landed in r46049.