Re: [PATCH v3 0/5] MR2103: Add support for WinRT app dark mode

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about include/windows.ui.viewmanagement.idl:
> -
>    );
> -
>    [eventremove] HRESULT TextScaleFactorChanged([in] EventRegistrationToken cookie);
> - }
> -
> - [
> -
>    contract(Windows.Foundation.UniversalApiContract, 1.0),
> -
>    exclusiveto(Windows.UI.ViewManagement.UISettings),
> -
>    uuid(03021be4-5254-4781-8194-5168f7d06d7b)
> - ]
> - interface IUISettings3 : IInspectable
> - {
> -
>    HRESULT GetColorValue([in] Windows.UI.ViewManagement.UIColorType color, [out, retval] Windows.UI.Color *value);
> -
>    [eventadd] HRESULT ColorValuesChanged(
> -
>        [in] Windows.Foundation.TypedEventHandler<Windows.UI.ViewManagement.UISettings *, IInspectable *> *handler,
> -
>        [out, retval] EventRegistrationToken *cookie
> -
>    );

I think we usually prefer a wrapping style like:

```c
        [eventadd] HRESULT ColorValuesChanged([in] Windows.Foundation.TypedEventHandler<Windows.UI.ViewManagement.UISettings *, IInspectable *> *handler,
                                              [out, retval] EventRegistrationToken *cookie);
```