Re: [2/3] winhttp: Implement IWinHttpRequest::get_ResponseBody. (try 2)

[Jacek Caban]

Hi Hans,

On 07/23/11 09:52, Hans Leidekker wrote:
> @@ -2107,7 +2107,7 @@ static void test_IWinHttpRequest(void)
>      HRESULT hr;
>      IWinHttpRequest *req;
>      BSTR method, url, response = NULL, status_text = NULL;
> - VARIANT async, empty, timeout;
> - VARIANT async, empty, timeout, body;
> VARIANT_BOOL succeeded;
> LONG status;
>
> @@ -2241,6 +2241,13 @@ static void test_IWinHttpRequest(void)
>      ok( hr == S_OK, "got %08x\n", hr );
>      SysFreeString( response );
> - VariantInit( &body );
> - V_VT( &body ) = VT_ERROR;
> - hr = IWinHttpRequest_get_ResponseBody( req, &body );
> - ok( hr == S_OK, "got %08x\n", hr );
> - ok( V_VT( &body ) == (VT_ARRAY|VT_UI1), "got %08x\n", V_VT( &body ) );
> -
> - ok( hr == S_OK, "got %08x\n", hr );

This duplicates the above test. Did you mean to test the result of
VariantClear here (otherwise there is a leak)?

Thanks,
    Jacek