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

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about include/windows.devices.geolocation.idl:
> -
>    exclusiveto(Windows.Devices.Geolocation.Geolocator),
> -
>    uuid(a9c3bf62-4524-4989-8aa9-de019d2e551f)
> - ]
> - interface IGeolocator : IInspectable
> - {
> -
>    [propget] HRESULT DesiredAccuracy([out, retval] Windows.Devices.Geolocation.PositionAccuracy *value);
> -
>    [propput] HRESULT DesiredAccuracy([in] Windows.Devices.Geolocation.PositionAccuracy value);
> -
>    [propget] HRESULT MovementThreshold([out, retval] DOUBLE *value);
> -
>    [propput] HRESULT MovementThreshold([in] DOUBLE value);
> -
>    [propget] HRESULT ReportInterval([out, retval] UINT32 *value);
> -
>    [propput] HRESULT ReportInterval([in] UINT32 value);
> -
>    [propget] HRESULT LocationStatus([out, retval] Windows.Devices.Geolocation.PositionStatus *value);
> -
>    [overload("GetGeopositionAsync")]
> -
>    HRESULT GetGeopositionAsync([out, retval] Windows.Foundation.IAsyncOperation<Windows.Devices.Geolocation.Geoposition*> **value);
> -
>    [overload("GetGeopositionAsync")]
> -
>    HRESULT GetGeopositionAsyncWithAgeAndTimeout([in] Windows.Foundation.TimeSpan maximumAge,

```suggestion:-0+0
        HRESULT GetGeopositionAsyncWithAgeAndTimeout([in] Windows.Foundation.TimeSpan maximum_age,
```