[Webkit-unassigned] [Bug 195999] Compilation broken in WKWebViewGetContents.mm with iOS SDK (rev 243195)

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=195999

--- Comment #9 from Frédéric Wang (:fredw) <fred.wang at free.fr> ---
(In reply to Timothy Hatcher from comment #8)
> (In reply to Frédéric Wang (:fredw) from comment #7)
> > (In reply to Frédéric Wang (:fredw) from comment #6)
> > > (In reply to Ali Juma from comment #2)
> > > > This typo (WK_MAC_TBA instead of WK_IOS_TBA) unfortunately happens every so
> > > > often and breaks the open-source iOS build.
> > > 
> > > I had added a style check in bug 181394 ; I wonder why the error was not
> > > caught by check-webkit-style?
> > 
> > OK it was actually detected:
> > https://bugs.webkit.org/show_bug.cgi?id=195636#c52
> 
> Oops, I remember seeing that and not fully understanding / reading it I
> guess.
> 
> "WK_MAC_TBA is neither a version number nor WK_IOS_TBA"
> 
> That statement is true, it is not a number nor WK_IOS_TBA — nothing to see
> here. It sounded (at a glance) like it didn't approve of a define being used
> — expecting only a number — since macosx(WK_MAC_TBA) was also used on the
> same line. The missing part is this is about the WK_MAC_TBA  being used in
> the ios() macro.
> 
> An error more like would help:
> 
> "ios(WK_MAC_TBA) is invalid; expected WK_IOS_TBA or a number"
> 
> I'll file a bug on this.

I see. Thanks.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20190321/7fc7df88/attachment-0001.html>