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

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.devices.geolocation.geolocator/main.c:
> -
>
> static HRESULT WINAPI factory_ActivateInstance(IActivationFactory *iface, IInspectable **instance)
>  {
> - FIXME("iface %p, instance %p stub!\n", iface, instance);
> - return E_NOTIMPL;
> - struct geolocator *impl;
> -
> - TRACE("iface %p, instance %p.\n", iface, instance);
> -
> - if (!(impl = calloc(1, sizeof(*impl))))
> - {
> -
>    *instance = NULL;
> -
>    return E_OUTOFMEMORY;
> - }
> -
> - impl->IActivationFactory_iface.lpVtbl = &factory_vtbl;

The `geolocator` class then should be separate from the `geolocator_statics`, and the geolocator instances don't implement `IActivationFactory`.