<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>