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:
> };
>
> +DEFINE_IINSPECTABLE(geolocator, IGeolocator, struct geolocator, IActivationFactory_iface)
> -
>
> +HRESULT WINAPI geolocator_DesiredAccuracy_get(IGeolocator *iface, PositionAccuracy *value)
> +{
> - FIXME("iface %p, value %p stub.\n", iface, value);
> - *value = 0;
> - return E_NOTIMPL;
>
> +}
> -
>
> +HRESULT WINAPI geolocator_DesiredAccuracy_set(IGeolocator *iface, PositionAccuracy value)
> +{
> - FIXME("iface %p, value %d stub.\n", iface, value);
> - return E_NOTIMPL;
>
> +}

We usually put get_ / set_ before the property name, to match the vtable function names.