Re: [PATCH v5 0/7] MR3189: Add windows.devices.geolocation.geolocator

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.devices.geolocation.geolocator/tests/geolocator.c:
> - ok(hr == S_OK, "got hr %#lx.\n", hr);
> -
> - hr = RoGetActivationFactory(str, &IID_IActivationFactory, (void **)&factory);
> - WindowsDeleteString(str);
> - ok(hr == S_OK || broken(hr == REGDB_E_CLASSNOTREG), "got hr %#lx.\n", hr);
> - if (hr == REGDB_E_CLASSNOTREG)
> - {
> -
>   win_skip("%s runtimeclass not registered, skipping tests.\n", wine_dbgstr_w(geolocator_name));
> -
>   return;
> - }
> -
> - check_interface(factory, &IID_IUnknown, FALSE);
> - check_interface(factory, &IID_IInspectable, FALSE);
> - check_interface(factory, &IID_IAgileObject, TRUE);
> - check_interface(factory, &IID_IActivationFactory, FALSE);
>
> +}

Although it doesn't really matter as it's static you're leaking the IActivationFactory reference here, please release it.