Re: [PATCH 0/16] MR4046: windows.media.mediacontrol: Implement some IMusicDisplayProperties and IMusicDisplayProperties2 properties.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.media.mediacontrol/display.c:
> static HRESULT WINAPI music_properties_get_Artist( IMusicDisplayProperties *iface, HSTRING *value )
>  {
> - FIXME( "iface %p, value %p stub!\n", iface, value );
> - return E_NOTIMPL;
> - struct music_properties *impl = impl_from_IMusicDisplayProperties( iface );
> - TRACE( "iface %p, value %p\n", iface, value );
> - return WindowsDuplicateString( impl->artist, value );
>
> }
>
> static HRESULT WINAPI music_properties_put_Artist( IMusicDisplayProperties *iface, HSTRING value )
>  {
> - FIXME( "iface %p, value %s stub!\n", iface, debugstr_hstring(value) );
> - return E_NOTIMPL;
> - struct music_properties *impl = impl_from_IMusicDisplayProperties( iface );
> - TRACE( "iface %p, value %p\n", iface, value );
> - return WindowsDuplicateString( value, &impl->artist );

Same here, you need to call `WindowsDeleteString` first.