<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 23, 2013 at 11:48 AM, Alistair Leslie-Hughes <span dir="ltr"><<a href="mailto:leslie_alistair@hotmail.com" target="_blank">leslie_alistair@hotmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
<br>
Changelog:<br>
    oledb32: GetConversionSize only returns a valid size of BSTR<br></blockquote><div><br> <pre>+            if(V_VT((VARIANT*)src) != VT_BSTR)
+                *dst_len = 110;
             else
-                return hr;
+                *dst_len = (SysStringLen(V_BSTR((VARIANT*)src)) + 1) * sizeof(WCHAR);
+
+            return S_OK;<br><br></pre><pre>So it returns previous result if type is invalid, meaning it's not VT_BSTR? You can't just return a random<br>constant that appears in tests just because of previously made call (which is my guess). If you really<br>
want to implement this broken behaviour it will need more tests, at least one more that shows it returns<br>a result from last successful call.<br></pre><pre><br><br></pre><pre> <br></pre></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<br>
<br>
Best Regards<span class=""><font color="#888888"><br>
 Alistair Leslie-Hughes<br>
</font></span><br><br>
<br></blockquote></div><br></div></div>