Re: [PATCH v13 0/5] MR8829: windows.media.playback.mediaplayer: Implement IActivationFactory::ActivateInstance().

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about include/windows.media.playback.idl:
> +    [
> +        contract(Windows.Foundation.UniversalApiContract, 1.0)
> +    ]
> +    enum MediaPlayerError
> +    {
> +        Unknown            = 0,
> +        Aborted            = 1,
> +        NetworkError       = 2,
> +        DecodingError      = 3,
> +        SourceNotSupported = 4,
> +    };
> +
> +    [
> +        contract(Windows.Foundation.UniversalApiContract, 1.0),
> +        /* deprecated("Use MediaPlaybackState instead of MediaPlayerState.  For more info, see MSDN.", deprecate, Windows.Foundation.UniversalApiContract, 3.0) */

Why commenting this? Also I don't think we need to match the message, just something like:

```suggestion:-0+0
        deprecated("Use MediaPlaybackState", deprecate, Windows.Foundation.UniversalApiContract, 3.0),
```

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8829#note_114509