Re: [PATCH 1/4] dmime: COM cleanup for IDirectMusicPerformance8.

[Christian Costa]

Le 02/04/2012 00:03, Michael Stefaniuc a écrit :
> ---
>
> -static HRESULT WINAPI IDirectMusicPerformance8Impl_MusicToReferenceTime (LPDIRECTMUSICPERFORMANCE8 iface, MUSIC_TIME mtTime, REFERENCE_TIME* prtTime) {
> -	IDirectMusicPerformance8Impl *This = (IDirectMusicPerformance8Impl *)iface;
> +static HRESULT WINAPI IDirectMusicPerformance8Impl_MusicToReferenceTime(IDirectMusicPerformance8 *iface,
> +        MUSIC_TIME mtTime, REFERENCE_TIME *prtTime)
> +{
> +        IDirectMusicPerformance8Impl *This = impl_from_IDirectMusicPerformance8(iface);
> +
>   FIXME("(%p, %d, %p): stub\n", This, mtTime, prtTime);
>   return S_OK;
>   }
>
> -static HRESULT WINAPI IDirectMusicPerformance8Impl_ReferenceToMusicTime (LPDIRECTMUSICPERFORMANCE8 iface, REFERENCE_TIME rtTime, MUSIC_TIME* pmtTime) {
> -	IDirectMusicPerformance8Impl *This = (IDirectMusicPerformance8Impl *)iface;
> +static HRESULT WINAPI IDirectMusicPerformance8Impl_ReferenceToMusicTime(IDirectMusicPerformance8 *iface,
> +        REFERENCE_TIME rtTime, MUSIC_TIME *pmtTime)
> +{
> +        IDirectMusicPerformance8Impl *This = impl_from_IDirectMusicPerformance8(iface);
> +
>   FIXME("(%p, 0x%s, %p): stub\n", This, wine_dbgstr_longlong(rtTime), pmtTime);
>   return S_OK;
>   }
>
> -static HRESULT WINAPI IDirectMusicPerformance8Impl_IsPlaying (LPDIRECTMUSICPERFORMANCE8 iface, IDirectMusicSegment* pSegment, IDirectMusicSegmentState* pSegState) {
> -	IDirectMusicPerformance8Impl *This = (IDirectMusicPerformance8Impl *)iface;
> +static HRESULT WINAPI IDirectMusicPerformance8Impl_IsPlaying(IDirectMusicPerformance8 *iface,
> +        IDirectMusicSegment *pSegment, IDirectMusicSegmentState *pSegState)
> +{
> +        IDirectMusicPerformance8Impl *This = impl_from_IDirectMusicPerformance8(iface);
> +
>   FIXME("(%p, %p, %p): stub\n", This, pSegment, pSegState);
>   return S_FALSE;
>   }

Hi Michael,

While you're at it, could you put a 4 spaces indentation to whole method.

Christian