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:
> -
> - typedef enum PositionAccuracy PositionAccuracy;
> - typedef enum PositionStatus PositionStatus;
> -
> - declare {
> -
>    interface Windows.Foundation.AsyncOperationCompletedHandler<Windows.Devices.Geolocation.Geoposition*>;
> -
>    interface Windows.Foundation.IAsyncOperation<Windows.Devices.Geolocation.Geoposition*>;
> -
>    interface Windows.Foundation.TypedEventHandler<Windows.Devices.Geolocation.Geolocator*, Windows.Devices.Geolocation.PositionChangedEventArgs*>;
> -
>    interface Windows.Foundation.TypedEventHandler<Windows.Devices.Geolocation.Geolocator*, Windows.Devices.Geolocation.StatusChangedEventArgs*>;
> - }
> -
> - [contract(Windows.Foundation.UniversalApiContract, 1.0)]
> - enum PositionAccuracy
> - {
> -
>    Default = 0,
> -
>    High    = 1

Could add a trailing comma to avoid unnecessary future line changes.