d3d10core: Add a separate function for d3d10_buffer initialization.
authorHenri Verbeet <hverbeet@codeweavers.com>
Thu, 17 Sep 2009 21:03:27 +0000 (23:03 +0200)
committerAlexandre Julliard <julliard@winehq.org>
Fri, 18 Sep 2009 13:54:52 +0000 (08:54 -0500)
dlls/d3d10core/buffer.c
dlls/d3d10core/d3d10core_private.h
dlls/d3d10core/device.c

index a4198c163517a8c17c6430ac65448ad4a68bbc8e..d27e1ea5b0a8cd032c732cf2f018ac424e8e2334 100644 (file)
@@ -145,7 +145,7 @@ static void STDMETHODCALLTYPE d3d10_buffer_GetDesc(ID3D10Buffer *iface, D3D10_BU
     FIXME("iface %p, desc %p stub!\n", iface, desc);
 }
 
-const struct ID3D10BufferVtbl d3d10_buffer_vtbl =
+static const struct ID3D10BufferVtbl d3d10_buffer_vtbl =
 {
     /* IUnknown methods */
     d3d10_buffer_QueryInterface,
@@ -165,3 +165,31 @@ const struct ID3D10BufferVtbl d3d10_buffer_vtbl =
     d3d10_buffer_Unmap,
     d3d10_buffer_GetDesc,
 };
+
+HRESULT d3d10_buffer_init(struct d3d10_buffer *buffer, struct d3d10_device *device,
+        const D3D10_BUFFER_DESC *desc, const D3D10_SUBRESOURCE_DATA *data)
+{
+    struct wined3d_buffer_desc wined3d_desc;
+    HRESULT hr;
+
+    buffer->vtbl = &d3d10_buffer_vtbl;
+    buffer->refcount = 1;
+
+    FIXME("Implement DXGI<->wined3d usage conversion\n");
+
+    wined3d_desc.byte_width = desc->ByteWidth;
+    wined3d_desc.usage = desc->Usage;
+    wined3d_desc.bind_flags = desc->BindFlags;
+    wined3d_desc.cpu_access_flags = desc->CPUAccessFlags;
+    wined3d_desc.misc_flags = desc->MiscFlags;
+
+    hr = IWineD3DDevice_CreateBuffer(device->wined3d_device, &wined3d_desc,
+            data ? data->pSysMem : NULL, (IUnknown *)buffer, &buffer->wined3d_buffer);
+    if (FAILED(hr))
+    {
+        WARN("Failed to create wined3d buffer, hr %#x.\n", hr);
+        return hr;
+    }
+
+    return S_OK;
+}
index 4257b2ea0beab02552a0e22baf5c4c04588c00bd..ad03b47a83991ad52cead7fed3676dc2da28eb66 100644 (file)
@@ -96,7 +96,6 @@ HRESULT d3d10_texture2d_init(struct d3d10_texture2d *texture, struct d3d10_devic
         const D3D10_TEXTURE2D_DESC *desc);
 
 /* ID3D10Buffer */
-extern const struct ID3D10BufferVtbl d3d10_buffer_vtbl;
 struct d3d10_buffer
 {
     const struct ID3D10BufferVtbl *vtbl;
@@ -105,6 +104,9 @@ struct d3d10_buffer
     IWineD3DBuffer *wined3d_buffer;
 };
 
+HRESULT d3d10_buffer_init(struct d3d10_buffer *buffer, struct d3d10_device *device,
+        const D3D10_BUFFER_DESC *desc, const D3D10_SUBRESOURCE_DATA *data);
+
 /* ID3D10RenderTargetView */
 extern const struct ID3D10RenderTargetViewVtbl d3d10_rendertarget_view_vtbl;
 struct d3d10_rendertarget_view
index 6b685b06b028255908c940e614207516bd262712..80cbc61b62721e8da03910d43e3792d7e6e7b520 100644 (file)
@@ -618,7 +618,6 @@ static HRESULT STDMETHODCALLTYPE d3d10_device_CreateBuffer(ID3D10Device *iface,
         const D3D10_BUFFER_DESC *desc, const D3D10_SUBRESOURCE_DATA *data, ID3D10Buffer **buffer)
 {
     struct d3d10_device *This = (struct d3d10_device *)iface;
-    struct wined3d_buffer_desc wined3d_desc;
     struct d3d10_buffer *object;
     HRESULT hr;
 
@@ -631,22 +630,10 @@ static HRESULT STDMETHODCALLTYPE d3d10_device_CreateBuffer(ID3D10Device *iface,
         return E_OUTOFMEMORY;
     }
 
-    object->vtbl = &d3d10_buffer_vtbl;
-    object->refcount = 1;
-
-    FIXME("Implement DXGI<->wined3d usage conversion\n");
-
-    wined3d_desc.byte_width = desc->ByteWidth;
-    wined3d_desc.usage = desc->Usage;
-    wined3d_desc.bind_flags = desc->BindFlags;
-    wined3d_desc.cpu_access_flags = desc->CPUAccessFlags;
-    wined3d_desc.misc_flags = desc->MiscFlags;
-
-    hr = IWineD3DDevice_CreateBuffer(This->wined3d_device, &wined3d_desc,
-            data ? data->pSysMem : NULL, (IUnknown *)object, &object->wined3d_buffer);
+    hr = d3d10_buffer_init(object, This, desc, data);
     if (FAILED(hr))
     {
-        ERR("CreateBuffer failed, returning %#x\n", hr);
+        WARN("Failed to initialize buffer, hr %#x.\n", hr);
         HeapFree(GetProcessHeap(), 0, object);
         return hr;
     }