<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 - [FreeType] Restore conditional compilation logic for recent HarfBuzz refactoring"
   href="https://bugs.webkit.org/show_bug.cgi?id=193036#c7">Comment # 7</a>
              on <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - [FreeType] Restore conditional compilation logic for recent HarfBuzz refactoring"
   href="https://bugs.webkit.org/show_bug.cgi?id=193036">bug 193036</a>
              from <span class="vcard"><a class="email" href="mailto:jmason@ibinx.com" title="Jim Mason <jmason@ibinx.com>"> <span class="fn">Jim Mason</span></a>
</span></b>
        <pre>Thanks for the quick review, Michael.

To summarize, !ENABLE(OPENTYPE_MATH) is the active part of this fix.  Without it, users of older versions of HarfBuzz encounter effectively dead code that does not compile.

USE(HARFBUZZ) was included in the fix, as it was part of the original logic.  Whether it is necessary or redundant in other situations, I cannot say.  However, I can say in this specific case, it is unnecessary.</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>