Re: [PATCH v3 0/4] MR8554: windows.media: Implement IActivationFactory::ActivateInstance() for IMediaTranscoder.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about include/windows.media.transcoding.idl:
> -
>    uuid(190c99d2-a0aa-4d34-86bc-eed1b12c2f5b)
> - ]
> - interface IMediaTranscoder : IInspectable
> - {
> -
>    [propput] HRESULT TrimStartTime([in] Windows.Foundation.TimeSpan value);
> -
>    [propget] HRESULT TrimStartTime([out, retval] Windows.Foundation.TimeSpan *value);
> -
>    [propput] HRESULT TrimStopTime([in] Windows.Foundation.TimeSpan value);
> -
>    [propget] HRESULT TrimStopTime([out, retval] Windows.Foundation.TimeSpan *value);
> -
>    [propput] HRESULT AlwaysReencode([in] boolean value);
> -
>    [propget] HRESULT AlwaysReencode([out, retval] boolean *value);
> -
>    [propput] HRESULT HardwareAccelerationEnabled([in] boolean value);
> -
>    [propget] HRESULT HardwareAccelerationEnabled([out, retval] boolean *value);
> -
>    HRESULT AddAudioEffect([in] HSTRING activatable_class_id);
> -
>    HRESULT AddAudioEffectWithSettings([in] HSTRING activatable_class_id, [in] boolean effect_required, [in] Windows.Foundation.Collections.IPropertySet *configuration);
> -
>    HRESULT AddVideoEffect([in] HSTRING activatable_class_id);
> -
>    HRESULT AddVideoEffectWithSettings([in] HSTRING activatable_class_id, [in] boolean effect_required, [in] Windows.Foundation.Collections.IPropertySet *configuration);

These four methods should have and `[overload("AddAudioEffect")]` attribute.