Bug 118373 - [Windows] Revise handling of CoreGraphics Include Files
Summary: [Windows] Revise handling of CoreGraphics Include Files
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows 7
: P2 Normal
Assignee: Brent Fulgham
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2013-07-03 16:44 PDT by Brent Fulgham
Modified: 2013-07-03 17:02 PDT (History)
3 users (show)

See Also:


Attachments
Patch (96.66 KB, patch)
2013-07-03 16:49 PDT, Brent Fulgham
andersca: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brent Fulgham 2013-07-03 16:44:50 PDT
Different version of the CoreGraphics library require different compiler features.  The VS2010 build does not provide some definitions needed by newer versions of CoreGraphics.  This patch allows VS2010 to build these newer sources.
Comment 1 Brent Fulgham 2013-07-03 16:49:26 PDT
Created attachment 206035 [details]
Patch
Comment 2 Radar WebKit Bug Importer 2013-07-03 16:50:15 PDT
<rdar://problem/14351106>
Comment 3 WebKit Commit Bot 2013-07-03 16:51:02 PDT
Attachment 206035 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCore/WebCorePrefix.h']" exit_code: 1
Source/WebCore/WebCorePrefix.h:146:  Alphabetical sorting problem.  [build/include_order] [4]
Source/WebCore/WebCorePrefix.h:147:  Alphabetical sorting problem.  [build/include_order] [4]
Total errors found: 2 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 Anders Carlsson 2013-07-03 16:53:01 PDT
Comment on attachment 206035 [details]
Patch

Looks good, but I don’t think you meant to include the whitespace changes in the Change Log.
Comment 5 Brent Fulgham 2013-07-03 17:02:42 PDT
Committed r152383: <http://trac.webkit.org/changeset/152383>