<div dir="ltr">Thank you for commenting.<div class="gmail_extra"><br><br><div class="gmail_quote">2014-07-03 1:31 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi,<br>
<br>
<br>
+Sub testSgnError(strings, error_num)<br>
+    on error resume next<br>
+<br>
+    Call Err.clear()<br>
+    Call Sgn(strings)<br>
+    Call ok(Err.number = error_num, "Err.number = " & Err.number)<br>
+End Sub<br>
+<br>
+Call testSgnError(Null, 445)<br>
This test fails on windows. Tests needs to pass both on windows and on wine.<br>
<br></blockquote><div>Yeah, it should fail. Sgn(Null) always returns error number 94 on windows. If we want Sgn(Null) to return the same error number on wine, we should use the right error const which is defined in  include/winerror.h when </div>

<div><span style="font-family:arial,sans-serif;font-size:14px">V_VT(arg) == VT_NULL. That means, </span><span style="font-family:arial,sans-serif;font-size:14px"> </span><span style="font-family:arial,sans-serif;font-size:14px">E_NOTIMPL is not the best one, we should find a replacement. </span></div>

<div><span style="font-family:arial,sans-serif;font-size:14px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:14px">Problem is, though I have tried every const from </span><font face="arial, sans-serif"><span style="font-size:14px">DISP_E_UNKNOWNINTERFACE</span></font><span style="font-size:14px;font-family:arial,sans-serif">to DISP_E_UNKNOWNINTERFACE, none of them is right. I have no idea what to do now.</span></div>

<div><span style="font-family:arial,sans-serif;font-size:14px"><br></span></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


+    if(V_VT(arg) == VT_NULL) {<br>
+        FIXME("Null argument not supported\n");<br>
+            return E_NOTIMPL;<br>
+    }<br>
Please fix the indentation here.<br>
<br>
Cheers,<br>
Piotr<br>
</blockquote></div><br></div></div>