<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2014-06-25 17:37 GMT+08:00 Piotr Caban <span dir="ltr"><<a href="mailto:piotr.caban@gmail.com" target="_blank">piotr.caban@gmail.com</a>></span>:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 06/25/14 11:35, Piotr Caban wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 06/25/14 04:21, Shuai Meng wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+    hres = VariantChangeType(&dst, v, VARIANT_LOCALBOOL, VT_R8);<br>
</blockquote>
There's no need to pass VARIANT_LOCALBOOL flag here.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  static HRESULT Global_Sgn(vbdisp_t *This, VARIANT *arg, unsigned<br>
args_cnt, VARIANT *res)<br>
  {<br>
-    FIXME("\n");<br>
-    return E_NOTIMPL;<br>
+    double v;<br>
+    HRESULT hres;<br>
+<br>
+    TRACE("(%s)\n", debugstr_variant(arg));<br>
+<br>
+    assert(args_cnt == 1);<br>
+<br>
+    hres = to_double(arg, &v);<br>
+    if (FAILED(hres))<br>
+        return S_OK;<br>
</blockquote></blockquote></div>
I haven't seen it earlier, you should probably return hres here.<br></blockquote><div>Yes, you are right, I made a mistake here. Thank you. </div></div><br></div></div>