<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [Win] Fix the wrong return value checking of ScriptItemize in UniscribeController::itemizeShapeAndPlace"
   href="https://bugs.webkit.org/show_bug.cgi?id=192805#c1">Comment # 1</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [Win] Fix the wrong return value checking of ScriptItemize in UniscribeController::itemizeShapeAndPlace"
   href="https://bugs.webkit.org/show_bug.cgi?id=192805">bug 192805</a>
              from <span class="vcard"><a class="email" href="mailto:Hironori.Fujii@sony.com" title="Fujii Hironori <Hironori.Fujii@sony.com>"> <span class="fn">Fujii Hironori</span></a>
</span></b>
        <pre>FAILED macro | Microsoft Docs
<a href="https://docs.microsoft.com/en-us/windows/desktop/api/winerror/nf-winerror-failed">https://docs.microsoft.com/en-us/windows/desktop/api/winerror/nf-winerror-failed</a>

<span class="quote">> #define FAILED(hr) (((HRESULT)(hr)) < 0)</span >

FAILED(rc) is always false.

ScriptItemize function | Microsoft Docs
<a href="https://docs.microsoft.com/en-us/windows/desktop/api/usp10/nf-usp10-scriptitemize">https://docs.microsoft.com/en-us/windows/desktop/api/usp10/nf-usp10-scriptitemize</a>

<span class="quote">> The function returns E_INVALIDARG if pwcInChars is set to NULL, cInChars is 0, pItems is set to NULL, or cMaxItems < 2.</span >

If I understand the code correctly, ScriptItemize never returns E_INVALIDARG in above code. There is no way to test.</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>