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

[Rémi Bernon]

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);
```

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2103#note_23233