Re: [PATCH v2 0/4] MR3519: windows.devices.enumeration: Implement IDeviceInformationStatics::CreateWatcherAqsFilter().

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about include/windows.devices.enumeration.idl:
> -
>    contract(Windows.Foundation.UniversalApiContract, 1.0),
> -
>    exclusiveto(Windows.Devices.Enumeration.DeviceInformation),
> -
>    uuid(c17f100e-3a46-4a78-8013-769dc9b97390)
> - ]
> - interface IDeviceInformationStatics : IInspectable
> - {
> -
>    [overload("CreateFromIdAsync")]
> -
>    HRESULT CreateFromIdAsync([in] HSTRING id,
> -
>            [out, retval] Windows.Foundation.IAsyncOperation<Windows.Devices.Enumeration.DeviceInformation *> **op);
> -
>    [overload("CreateFromIdAsync")]
> -
>    HRESULT CreateFromIdAsyncAdditionalProperties([in] HSTRING id,
> -
>            [in] Windows.Foundation.Collections.IIterable<HSTRING> *additional_properties,
> -
>            [out, retval] Windows.Foundation.IAsyncOperation<Windows.Devices.Enumeration.DeviceInformation *> **op);
> -
>    [overload("FindAllAsync")]
> -
>    HRESULT FindAllAsync([out, retval] Windows.Foundation.IAsyncOperation<Windows.Devices.Enumeration.DeviceInformationCollection *> **op);
> -
>    [default_overload] [overload("FindAllAsync")]

I'd use the same attribute style as everywhere else (and put overload first as it's more common):

```suggestion:-0+0
        [overload("FindAllAsync"), default_overload]
```