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:
>  geolocator_StatusChanged_remove,
>
> };
>
> +static inline struct geolocator *impl_from_IWeakReference(IWeakReference *iface)
> +{
> - return CONTAINING_RECORD(iface, struct geolocator, IWeakReference_iface);
>
> +}
> -
>
> +static HRESULT WINAPI weakreference_QueryInterface(IWeakReference *iface, REFIID iid, void **out)

```suggestion:-0+0
static HRESULT WINAPI weak_reference_QueryInterface(IWeakReference *iface, REFIID iid, void **out)
```

Would be more readable imho, similarly for the other functions below.