Re: [PATCH v13 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 ULONG WINAPI weak_reference_Release(IWeakReference *iface)
> +{
> - struct geolocator *impl = impl_from_IWeakReference(iface);
> - ULONG ref = InterlockedDecrement(&impl->ref_weak);
> - if (!ref)
> -
>    free(impl);
> - return ref;
>
> +}
> -
>
> +static HRESULT WINAPI weak_reference_Resolve(IWeakReference *iface, REFIID iid, IInspectable **out)
> +{
> - HRESULT hr;
> - LONG ref;
> - struct geolocator *impl = impl_from_IWeakReference(iface);
> - TRACE("iface %p, iid %s, out %p stub.\n", iface, debugstr_guid(iid), out);

```suggestion:-3+0
    struct geolocator *impl = impl_from_IWeakReference(iface);
    HRESULT hr;
    LONG ref;

TRACE("iface %p, iid %s, out %p stub.\n", iface, debugstr_guid(iid), out);
```