[PATCH 4/5] d3dx9_36: Implement ID3DXFileEnumObjectImpl_GetChildren & ID3DXFileEnumObjectImpl_GetChild and add stubbed interface for ID3DXFileData. (try 6)

Christian Costa titan.costa at gmail.com
Thu Oct 25 16:17:16 CDT 2012


---
 dlls/d3dx9_36/xfile.c |  224 ++++++++++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 218 insertions(+), 6 deletions(-)

diff --git a/dlls/d3dx9_36/xfile.c b/dlls/d3dx9_36/xfile.c
index ae0eda1..074daf1 100644
--- a/dlls/d3dx9_36/xfile.c
+++ b/dlls/d3dx9_36/xfile.c
@@ -35,8 +35,17 @@ typedef struct {
 typedef struct {
     ID3DXFileEnumObject ID3DXFileEnumObject_iface;
     LONG ref;
+    ULONG nb_children;
+    ID3DXFileData **children;
 } ID3DXFileEnumObjectImpl;
 
+typedef struct {
+    ID3DXFileData ID3DXFileData_iface;
+    LONG ref;
+    IDirectXFileData *dxfile_data;
+} ID3DXFileDataImpl;
+
+
 static inline ID3DXFileImpl* impl_from_ID3DXFile(ID3DXFile *iface)
 {
     return CONTAINING_RECORD(iface, ID3DXFileImpl, ID3DXFile_iface);
@@ -47,6 +56,171 @@ static inline ID3DXFileEnumObjectImpl* impl_from_ID3DXFileEnumObject(ID3DXFileEn
     return CONTAINING_RECORD(iface, ID3DXFileEnumObjectImpl, ID3DXFileEnumObject_iface);
 }
 
+static inline ID3DXFileDataImpl* impl_from_ID3DXFileData(ID3DXFileData *iface)
+{
+    return CONTAINING_RECORD(iface, ID3DXFileDataImpl, ID3DXFileData_iface);
+}
+
+/*** IUnknown methods ***/
+
+static HRESULT WINAPI ID3DXFileDataImpl_QueryInterface(ID3DXFileData *iface, REFIID riid, void **ret_iface)
+{
+    TRACE("(%p)->(%s, %p)\n", iface, debugstr_guid(riid), ret_iface);
+
+    if (IsEqualGUID(riid, &IID_IUnknown) ||
+        IsEqualGUID(riid, &IID_ID3DXFileData))
+    {
+        iface->lpVtbl->AddRef(iface);
+        *ret_iface = iface;
+        return S_OK;
+    }
+
+    WARN("(%p)->(%s, %p), not found\n", iface, debugstr_guid(riid), ret_iface);
+    *ret_iface = NULL;
+    return E_NOINTERFACE;
+}
+
+static ULONG WINAPI ID3DXFileDataImpl_AddRef(ID3DXFileData *iface)
+{
+    ID3DXFileDataImpl *This = impl_from_ID3DXFileData(iface);
+    ULONG ref = InterlockedIncrement(&This->ref);
+
+    TRACE("(%p)->(): new ref %d\n", iface, ref);
+
+    return ref;
+}
+
+static ULONG WINAPI ID3DXFileDataImpl_Release(ID3DXFileData *iface)
+{
+    ID3DXFileDataImpl *This = impl_from_ID3DXFileData(iface);
+    ULONG ref = InterlockedDecrement(&This->ref);
+
+    TRACE("(%p)->(): new ref %d\n", iface, ref);
+
+    if (!ref)
+    {
+        IDirectXFileData_Release(This->dxfile_data);
+        HeapFree(GetProcessHeap(), 0, This);
+    }
+
+    return ref;
+}
+
+
+/*** ID3DXFileData methods ***/
+
+static HRESULT WINAPI ID3DXFileDataImpl_GetEnum(ID3DXFileData *iface, ID3DXFileEnumObject **enum_object)
+{
+    FIXME("(%p)->(%p): stub\n", iface, enum_object);
+
+    return E_NOTIMPL;
+}
+
+
+static HRESULT WINAPI ID3DXFileDataImpl_GetName(ID3DXFileData *iface, char *name, SIZE_T *size)
+{
+    FIXME("(%p)->(%p, %p): stub\n", iface, name, size);
+
+    return E_NOTIMPL;
+}
+
+
+static HRESULT WINAPI ID3DXFileDataImpl_GetId(ID3DXFileData *iface, GUID *guid)
+{
+    FIXME("(%p)->(%p): stub\n", iface, guid);
+
+    return E_NOTIMPL;
+}
+
+
+static HRESULT WINAPI ID3DXFileDataImpl_Lock(ID3DXFileData *iface, SIZE_T *size, const void **data)
+{
+    FIXME("(%p)->(%p, %p): stub\n", iface, size, data);
+
+    return E_NOTIMPL;
+}
+
+
+static HRESULT WINAPI ID3DXFileDataImpl_Unlock(ID3DXFileData *iface)
+{
+    FIXME("(%p)->(): stub\n", iface);
+
+    return E_NOTIMPL;
+}
+
+
+static HRESULT WINAPI ID3DXFileDataImpl_GetType(ID3DXFileData *iface, GUID *guid)
+{
+    FIXME("(%p)->(%p): stub\n", iface, guid);
+
+    return E_NOTIMPL;
+}
+
+
+static BOOL WINAPI ID3DXFileDataImpl_IsReference(ID3DXFileData *iface)
+{
+    TRACE("(%p)->(): stub\n", iface);
+
+    return E_NOTIMPL;
+}
+
+
+static HRESULT WINAPI ID3DXFileDataImpl_GetChildren(ID3DXFileData *iface, SIZE_T *children)
+{
+    TRACE("(%p)->(%p): stub\n", iface, children);
+
+    return E_NOTIMPL;
+}
+
+
+static HRESULT WINAPI ID3DXFileDataImpl_GetChild(ID3DXFileData *iface, SIZE_T id, ID3DXFileData **object)
+{
+    TRACE("(%p)->(%lu, %p): stub\n", iface, id, object);
+
+    return E_NOTIMPL;
+}
+
+
+static const ID3DXFileDataVtbl ID3DXFileData_Vtbl =
+{
+    ID3DXFileDataImpl_QueryInterface,
+    ID3DXFileDataImpl_AddRef,
+    ID3DXFileDataImpl_Release,
+    ID3DXFileDataImpl_GetEnum,
+    ID3DXFileDataImpl_GetName,
+    ID3DXFileDataImpl_GetId,
+    ID3DXFileDataImpl_Lock,
+    ID3DXFileDataImpl_Unlock,
+    ID3DXFileDataImpl_GetType,
+    ID3DXFileDataImpl_IsReference,
+    ID3DXFileDataImpl_GetChildren,
+    ID3DXFileDataImpl_GetChild
+};
+
+
+static HRESULT ID3DXFileDataImpl_Create(IDirectXFileObject *dxfile_object, ID3DXFileData **ret_iface)
+{
+    ID3DXFileDataImpl *object;
+
+    TRACE("(%p, %p)\n", dxfile_object, ret_iface);
+
+    *ret_iface = NULL;
+
+    object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*object));
+    if (!object)
+        return E_OUTOFMEMORY;
+
+    object->ID3DXFileData_iface.lpVtbl = &ID3DXFileData_Vtbl;
+    object->ref = 1;
+
+    /* FIXME: For now we are only called by enum object with FileData object. Later it will be objects derived from FileObject */
+    object->dxfile_data = (IDirectXFileData*)dxfile_object;
+
+    *ret_iface = &object->ID3DXFileData_iface;
+
+    return S_OK;
+}
+
 
 /*** IUnknown methods ***/
 
@@ -85,7 +259,14 @@ static ULONG WINAPI ID3DXFileEnumObjectImpl_Release(ID3DXFileEnumObject *iface)
     TRACE("(%p)->(): new ref %d\n", iface, ref);
 
     if (!ref)
+    {
+        ULONG i;
+
+        for (i = 0; i < This->nb_children; i++)
+            (This->children[i])->lpVtbl->Release(This->children[i]);
+        HeapFree(GetProcessHeap(), 0, This->children);
         HeapFree(GetProcessHeap(), 0, This);
+    }
 
     return ref;
 }
@@ -103,17 +284,26 @@ static HRESULT WINAPI ID3DXFileEnumObjectImpl_GetFile(ID3DXFileEnumObject *iface
 
 static HRESULT WINAPI ID3DXFileEnumObjectImpl_GetChildren(ID3DXFileEnumObject *iface, SIZE_T *children)
 {
-    FIXME("(%p)->(%p): stub\n", iface, children);
+    ID3DXFileEnumObjectImpl *This = impl_from_ID3DXFileEnumObject(iface);
 
-    return E_NOTIMPL;
+    TRACE("(%p)->(%p)\n", iface, children);
+
+    *children = This->nb_children;
+
+    return S_OK;
 }
 
 
 static HRESULT WINAPI ID3DXFileEnumObjectImpl_GetChild(ID3DXFileEnumObject *iface, SIZE_T id, ID3DXFileData **object)
 {
-    FIXME("(%p)->(%lu, %p): stub\n", iface, id, object);
+    ID3DXFileEnumObjectImpl *This = impl_from_ID3DXFileEnumObject(iface);
 
-    return E_NOTIMPL;
+    TRACE("(%p)->(%lu, %p)\n", iface, id, object);
+
+    *object = This->children[id];
+    (*object)->lpVtbl->AddRef(*object);
+
+    return S_OK;
 }
 
 
@@ -205,9 +395,10 @@ static HRESULT WINAPI ID3DXFileImpl_CreateEnumObject(ID3DXFile *iface, const voi
     DXFILELOADOPTIONS dxfile_options;
     DXFILELOADRESOURCE dxfile_resource;
     DXFILELOADMEMORY dxfile_memory;
+    IDirectXFileData *data_object;
     HRESULT ret;
 
-    FIXME("(%p)->(%p, %x, %p): partial stub\n", iface, source, options, enum_object);
+    TRACE("(%p)->(%p, %x, %p)\n", iface, source, options, enum_object);
 
     *enum_object = NULL;
 
@@ -256,10 +447,31 @@ static HRESULT WINAPI ID3DXFileImpl_CreateEnumObject(ID3DXFile *iface, const voi
         return ret;
     }
 
-    /* FIXME: Fill enum object with top level data objects */
+    /* Fill enum object with top level data objects */
+    while (SUCCEEDED(ret = IDirectXFileEnumObject_GetNextDataObject(dxfile_enum_object, &data_object)))
+    {
+        if (object->children)
+            object->children = HeapReAlloc(GetProcessHeap(), 0, object->children, sizeof(*object->children) * (object->nb_children + 1));
+        else
+            object->children = HeapAlloc(GetProcessHeap(), 0, sizeof(*object->children));
+        if (!object->children)
+        {
+            ret = E_OUTOFMEMORY;
+            break;
+        }
+        ret = ID3DXFileDataImpl_Create((IDirectXFileObject*)data_object, &object->children[object->nb_children]);
+        if (ret != S_OK)
+            break;
+        object->nb_children++;
+    }
 
     IDirectXFileEnumObject_Release(dxfile_enum_object);
 
+    if (ret != DXFILEERR_NOMOREOBJECTS)
+        WARN("Cannot get all top level data objects\n");
+
+    TRACE("Found %u children\n", object->nb_children);
+
     *enum_object = &object->ID3DXFileEnumObject_iface;
 
     return S_OK;




More information about the wine-patches mailing list