[PATCH 5/5] adsldp/tests: Add some tests for LDAPNamespace.

Sven Baars sven.wine at gmail.com
Mon Dec 16 08:11:58 CST 2019


On 16-12-2019 15:05, Dmitry Timoshkov wrote:
> Sven Baars <sven.wine at gmail.com> wrote:
> 
>>> +START_TEST(ldap)
>>> +{
>>> +    HRESULT hr;
>>> +
>>> +    hr = CoInitialize(NULL);
>>> +    ok(hr == S_OK, "got %#x\n", hr);
>>> +
>>> +    test_LDAP();
>>> +}
>>>
>> I think you forgot to call CoUninitialize here.
> 
> The process is going to terminate anyway, so it's a waste of time.
> 
In that case, you might as well remove all *_Release calls from your
tests...



More information about the wine-devel mailing list