Bug 174299

Summary: [Xcode] ICU headers aren’t treated as system headers after r219155
Product: WebKit Reporter: mitz
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, keith_miller, mark.lam, mmaxfield, msaboff, saam, sam
Priority: P2    
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=116407
Attachments:
Description Flags
Treat ICU headers as system headers sam: review+

mitz
Reported 2017-07-09 12:35:49 PDT
<https://trac.webkit.org/changeset/219155/webkit/trunk/Source/JavaScriptCore> made it so that the ICU headers were no longer treated as system headers, and tried to compensate by suppressing the documentation warning around each #include of an ICU header that triggered the warning. Instead, we can keep treating the headers as system headers. Patch forthcoming.
Attachments
Treat ICU headers as system headers (4.20 KB, patch)
2017-07-09 12:37 PDT, mitz 		sam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2017-07-09 12:37:57 PDT
Created attachment 314951 [details] Treat ICU headers as system headers
mitz
Comment 2 2017-07-09 13:19:58 PDT
Fixed in <https://trac.webkit.org/r219282>.
Note You need to log in before you can comment on or make changes to this bug.