[1/2] msxml3/httprequest: Store passed data for IXMLHttpRequest::open()

Jacek Caban jacek at codeweavers.com
Tue Sep 28 06:51:59 CDT 2010


  Hi Nikolay,

On 9/28/10 1:30 PM, Nikolay Sivov wrote:
>  Resent for series completeness.

+    hr = VariantChangeType(&str,&user, 0, VT_BSTR);
+    if (hr == S_OK)
+    {
+        This->user = SysAllocString(V_BSTR(&str));
+        VariantClear(&str);
+    }

You already own the reference to string in str VARIANT, so you can just use it |This->user = V_BSTR(&str);| and remove VariantClear call.


+    hr = VariantChangeType(&str,&password, 0, VT_BSTR);
+    if (hr == S_OK)
+    {
+        This->password = SysAllocString(V_BSTR(&str));
+        VariantClear(&str);
+    }

Same here.


Jacek




More information about the wine-devel mailing list