[2/3] winhttp: Implement IWinHttpRequest::get_ResponseBody.

Jacek Caban jacek at codeweavers.com
Fri Jul 22 09:42:41 CDT 2011


On 07/22/11 16:32, Hans Leidekker wrote:
> On Fri, 2011-07-22 at 16:16 +0200, Jacek Caban wrote:
>
>> The caller is responsible for freeing the result, so it will destroy
>> your internal data. I think you should create an array here and not use
>> it to store the data internally.
> How do you know that the caller is responsible for freeing this
> property?

That's how out and retval values work in OLE automation, you wouldn't be
able to control their life time otherwise. As a test, you may add
VariantClear call in your tests and see how things will work.

>  It seems wasteful to duplicate this potentially large buffer.

I agree, but we can't redesign the API.

Jacek



More information about the wine-devel mailing list