<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - Compilation broken in WKWebViewGetContents.mm with iOS SDK (rev 243195)"
   href="https://bugs.webkit.org/show_bug.cgi?id=195999#c8">Comment # 8</a>
              on <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - Compilation broken in WKWebViewGetContents.mm with iOS SDK (rev 243195)"
   href="https://bugs.webkit.org/show_bug.cgi?id=195999">bug 195999</a>
              from <span class="vcard"><a class="email" href="mailto:timothy@apple.com" title="Timothy Hatcher <timothy@apple.com>"> <span class="fn">Timothy Hatcher</span></a>
</span></b>
        <pre>(In reply to Frédéric Wang (:fredw) from <a href="show_bug.cgi?id=195999#c7">comment #7</a>)
<span class="quote">> (In reply to Frédéric Wang (:fredw) from <a href="show_bug.cgi?id=195999#c6">comment #6</a>)
> > (In reply to Ali Juma from <a href="show_bug.cgi?id=195999#c2">comment #2</a>)
> > > 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 <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - check-webkit-style: Verify syntax for WK_API_AVAILABLE"
   href="show_bug.cgi?id=181394">bug 181394</a> ; I wonder why the error was not
> > caught by check-webkit-style?

> OK it was actually detected:
> <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - Add new NSAttributedString API for converting HTML"
   href="show_bug.cgi?id=195636#c52">https://bugs.webkit.org/show_bug.cgi?id=195636#c52</a></span >

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.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>