ole32: Implement semi-stub for CoGetApartmentType.

Nikolay Sivov bunglehead at gmail.com
Wed Mar 2 02:00:53 CST 2016


On 02.03.2016 10:58, Sebastian Lackner wrote:
> On 02.03.2016 08:40, Nikolay Sivov wrote:
>> On 02.03.2016 10:30, Sebastian Lackner wrote:
>>> +    return info->apt ? ERROR_SUCCESS : CO_E_NOTINITIALIZED;
>>
>> It should be S_OK.
> 
> Oops, good catch. Thanks.
> 
>>
>>> +    hr = pCoGetApartmentType(NULL, NULL);
>>> +    ok(hr == E_INVALIDARG, "CoGetApartmentType succeeded, error: 0x%08x\n", hr);
>>> +    hr = pCoGetApartmentType(&type, NULL);
>>> +    ok(hr == E_INVALIDARG, "CoGetApartmentType succeeded, error: 0x%08x\n", hr);
>>> +    hr = pCoGetApartmentType(NULL, &qualifier);
>>> +    ok(hr == E_INVALIDARG, "CoGetApartmentType succeeded, error: 0x%08x\n", hr);
>>> +
>>> +    hr = pCoGetApartmentType(&type, &qualifier);
>>> +    ok(hr == CO_E_NOTINITIALIZED, "CoGetApartmentType succeeded, error: 0x%08x\n", hr);
>>> +    ok(type == APTTYPE_CURRENT, "Expected APTTYPE_CURRENT, got %u\n", type);
>>> +    ok(qualifier == APTTYPEQUALIFIER_NONE, "Expected APTTYPEQUALIFIER_NONE, got %u\n", qualifier);
>>
>> Please set out variables to some unrealistic values before the calls, to
>> see if it actually sets them.
>>
>> Also if they are set for CO_E_NOTINITIALIZED, it's possible they are set
>> for E_INVALIDARG cases too, and that's not tested (for both initialized
>> and not initialized cases).
> 
> I've added some variable initializations and tests for the E_INVALIDARG case.
> (Well, and I forgot to adjust the subject when sending v2, but that shouldn't matter ^^)
> 
> Does it look good for you now, or any additional remarks?
> 

No, nothing else from me.



More information about the wine-devel mailing list