Re: [PATCH 5/5] d3dx9_36: Implement ID3DXFileDataImpl_GetType. (try 5)

[Nikolay Sivov]

On 10/25/2012 10:34, Christian Costa wrote:
> dlls/d3dx9_36/xfile.c |   14 ++++++++++++--
>   1 file changed, 12 insertions(+), 2 deletions(-)
>
> diff --git a/dlls/d3dx9_36/xfile.c b/dlls/d3dx9_36/xfile.c
> index a1e0596..4c216da 100644
> --- a/dlls/d3dx9_36/xfile.c
> +++ b/dlls/d3dx9_36/xfile.c
> @@ -154,9 +154,19 @@ static HRESULT WINAPI ID3DXFileDataImpl_Unlock(ID3DXFileData *iface)
>
> static HRESULT WINAPI ID3DXFileDataImpl_GetType(ID3DXFileData *iface, GUID *guid)
>   {
> - FIXME("(%p)->(%p): stub\n", iface, guid);
> - ID3DXFileDataImpl *This = impl_from_ID3DXFileData(iface);
> - const GUID *dxfile_guid;
> - HRESULT ret;
> - return E_NOTIMPL;
> - TRACE("(%p)->(%p)\n", iface, guid);
> -
> - ret = IDirectXFileData_GetType(This->dxfile_object, &dxfile_guid);
> - if (ret != S_OK)
> -
>    return ret;
> -
> - *guid = *dxfile_guid;
> -
> - return S_OK;
> }

Why not simply forward to  IDirectXFileData here? What is all the 
surrounding code for?