Bug 164048

Summary: [Xcode] WebKit fails to build when CLANG_WARN_DOCUMENTATION_COMMENTS is enabled
Product: WebKit Reporter: mitz
Component: WebKit APIAssignee: mitz
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, ossy, sam, thorton
Priority: P2    
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Enable and address documentation warnings
none
An attempt to reenable the warning for macOS and iOS none

mitz
Reported 2016-10-26 18:35:59 PDT
The compiler doesn’t like some of the documentation comments in the headers.
Attachments
Enable and address documentation warnings (29.61 KB, patch)
2016-10-26 21:40 PDT, mitz 		no flags
DetailsFormatted DiffDiff
An attempt to reenable the warning for macOS and iOS (3.39 KB, patch)
2016-10-27 00:53 PDT, mitz 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2016-10-26 21:40:00 PDT
Created attachment 292991 [details] Enable and address documentation warnings
WebKit Commit Bot
Comment 2 2016-10-26 22:20:25 PDT
Comment on attachment 292991 [details] Enable and address documentation warnings Clearing flags on attachment: 292991 Committed r207934: <http://trac.webkit.org/changeset/207934>
WebKit Commit Bot
Comment 3 2016-10-26 22:20:29 PDT
All reviewed patches have been landed. Closing bug.
mitz
Comment 4 2016-10-26 22:49:40 PDT
Tried to fix those iOS build errors in <https://trac.webkit.org/r207935>.
Csaba Osztrogonác
Comment 5 2016-10-27 00:17:25 PDT
(In reply to comment #4) > Tried to fix those iOS build errors in <https://trac.webkit.org/r207935>. This buildfix broke the iOS build, see build.webkit.org for details.
mitz
Comment 6 2016-10-27 00:32:09 PDT
Thanks. I reverted the configuration changes in r207951 until I can find a fix.
mitz
Comment 7 2016-10-27 00:53:46 PDT
Created attachment 292999 [details] An attempt to reenable the warning for macOS and iOS
mitz
Comment 8 2016-10-27 07:13:48 PDT
Reenabled in <https://trac.webkit.org/r207990>.
Note You need to log in before you can comment on or make changes to this bug.