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

[Rémi Bernon (＠rbernon)]

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