April 2014 - Wine-commits - list.winehq.org

Victor Martinez Calvo : msvcrt: Avoid null dereference in TRACE in MSVCRT__wfreopen.
by Alexandre Julliard 30 Apr '14

30 Apr '14

Module: wine Branch: master Commit: f6bb1a51a96fb550f9d5cc4f208364c11147c9e7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f6bb1a51a96fb550f9d5cc4f2… Author: Victor Martinez Calvo <vmartinez(a)reactos.org> Date: Tue Apr 29 17:43:26 2014 +0200 msvcrt: Avoid null dereference in TRACE in MSVCRT__wfreopen. --- dlls/msvcrt/file.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/msvcrt/file.c b/dlls/msvcrt/file.c index aff5a89..957cfaf 100644 --- a/dlls/msvcrt/file.c +++ b/dlls/msvcrt/file.c @@ -4040,7 +4040,7 @@ MSVCRT_FILE* CDECL MSVCRT__wfreopen(const MSVCRT_wchar_t *path, const MSVCRT_wch { int open_flags, stream_flags, fd; - TRACE(":path (%s) mode (%s) file (%p) fd (%d)\n", debugstr_w(path), debugstr_w(mode), file, file->_file); + TRACE(":path (%s) mode (%s) file (%p) fd (%d)\n", debugstr_w(path), debugstr_w(mode), file, file ? file->_file : -1); LOCK_FILES(); if (!file || ((fd = file->_file) < 0) || fd > MSVCRT_fdend)

1 0

Frédéric Delanoy : Assorted spelling/ grammar fixes.
by Alexandre Julliard 30 Apr '14

30 Apr '14

Module: wine Branch: master Commit: 2e5cb7aac869fab43f76f267a77d72b3384c7d2d URL: http://source.winehq.org/git/wine.git/?a=commit;h=2e5cb7aac869fab43f76f267a… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Wed Apr 30 15:42:10 2014 +0200 Assorted spelling/grammar fixes. --- dlls/comctl32/imagelist.c | 6 +++--- dlls/comdlg32/filedlg.c | 2 +- dlls/d3dx9_36/tests/math.c | 2 +- dlls/ddraw/device.c | 8 ++++---- dlls/dmime/segment.c | 2 +- dlls/gdi32/bitmap.c | 2 +- dlls/gphoto2.ds/gphoto2_main.c | 2 +- dlls/imm32/tests/imm32.c | 2 +- dlls/inetmib1/tests/main.c | 2 +- dlls/kernel32/tests/sync.c | 2 +- dlls/mstask/tests/task_scheduler.c | 2 +- dlls/ole32/compobj.c | 4 ++-- dlls/oleaut32/tests/safearray.c | 2 +- dlls/quartz/regsvr.c | 2 +- dlls/riched20/reader.c | 2 +- dlls/shell32/tests/shlfileop.c | 12 ++++++------ 16 files changed, 27 insertions(+), 27 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=2e5cb7aac869fab43f76f…

1 0

Vincent Povirk : gdiplus/tests: Fix image leak in graphics test.
by Alexandre Julliard 30 Apr '14

30 Apr '14

Module: wine Branch: master Commit: efc1d791a38aca1d5bb0095e9bbcef3c4becc934 URL: http://source.winehq.org/git/wine.git/?a=commit;h=efc1d791a38aca1d5bb0095e9… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Tue Apr 29 13:08:02 2014 -0500 gdiplus/tests: Fix image leak in graphics test. --- dlls/gdiplus/tests/graphics.c | 27 ++++++++++++++++++--------- 1 file changed, 18 insertions(+), 9 deletions(-) diff --git a/dlls/gdiplus/tests/graphics.c b/dlls/gdiplus/tests/graphics.c index 9fb1347..bd227dc 100644 --- a/dlls/gdiplus/tests/graphics.c +++ b/dlls/gdiplus/tests/graphics.c @@ -92,7 +92,7 @@ static REAL units_scale(GpUnit from, GpUnit to, REAL dpi) return pixels_to_units(pixels, to, dpi); } -static GpGraphics *create_graphics(REAL res_x, REAL res_y, GpUnit unit, REAL scale) +static GpGraphics *create_graphics(REAL res_x, REAL res_y, GpUnit unit, REAL scale, GpImage **image) { GpStatus status; union @@ -117,11 +117,8 @@ static GpGraphics *create_graphics(REAL res_x, REAL res_y, GpUnit unit, REAL sca status = GdipGetImageGraphicsContext(u.image, &graphics); expect(Ok, status); - /* image is intentionally leaked to make sure that there is no - side effects after its destruction. - status = GdipDisposeImage(u.image); - expect(Ok, status); - */ + + *image = u.image; status = GdipGetDpiX(graphics, &res); expect(Ok, status); @@ -3519,7 +3516,9 @@ static void test_GdipMeasureString(void) for (i = 0; i < sizeof(td)/sizeof(td[0]); i++) { - graphics = create_graphics(td[i].res_x, td[i].res_y, td[i].unit, td[i].page_scale); + GpImage *image; + + graphics = create_graphics(td[i].res_x, td[i].res_y, td[i].unit, td[i].page_scale, &image); lf.lfHeight = 0xdeadbeef; status = GdipGetLogFontW(font, graphics, &lf); @@ -3570,6 +3569,9 @@ todo_wine status = GdipDeleteGraphics(graphics); expect(Ok, status); + + status = GdipDisposeImage(image); + expect(Ok, status); } GdipDeleteFont(font); @@ -3593,8 +3595,9 @@ todo_wine for (i = 0; i < sizeof(td)/sizeof(td[0]); i++) { REAL unit_scale; + GpImage *image; - graphics = create_graphics(td[i].res_x, td[i].res_y, td[i].unit, td[i].page_scale); + graphics = create_graphics(td[i].res_x, td[i].res_y, td[i].unit, td[i].page_scale, &image); lf.lfHeight = 0xdeadbeef; status = GdipGetLogFontW(font, graphics, &lf); @@ -3664,6 +3667,9 @@ todo_wine status = GdipDeleteGraphics(graphics); expect(Ok, status); + + status = GdipDisposeImage(image); + expect(Ok, status); } GdipDeleteFont(font); @@ -3701,12 +3707,13 @@ static void test_transform(void) }; GpStatus status; GpGraphics *graphics; + GpImage *image; GpPointF ptf[2]; UINT i; for (i = 0; i < sizeof(td)/sizeof(td[0]); i++) { - graphics = create_graphics(td[i].res_x, td[i].res_y, td[i].unit, td[i].scale); + graphics = create_graphics(td[i].res_x, td[i].res_y, td[i].unit, td[i].scale, &image); ptf[0].X = td[i].in[0].X; ptf[0].Y = td[i].in[0].Y; ptf[1].X = td[i].in[1].X; @@ -3725,6 +3732,8 @@ static void test_transform(void) expectf(td[i].in[1].Y, ptf[1].Y); status = GdipDeleteGraphics(graphics); expect(Ok, status); + status = GdipDisposeImage(image); + expect(Ok, status); } }

1 0

Henri Verbeet : ddraw/tests: Handle the Windows 8 testbot in test_primary_palette().
by Alexandre Julliard 30 Apr '14

30 Apr '14

Module: wine Branch: master Commit: 174dbaf0f338a4174577ef38c88cb3ab931f92a0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=174dbaf0f338a4174577ef38c… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Apr 29 22:02:45 2014 +0200 ddraw/tests: Handle the Windows 8 testbot in test_primary_palette(). --- dlls/ddraw/tests/ddraw1.c | 17 +++++++++++++++++ dlls/ddraw/tests/ddraw2.c | 17 +++++++++++++++++ dlls/ddraw/tests/ddraw4.c | 17 +++++++++++++++++ dlls/ddraw/tests/ddraw7.c | 17 +++++++++++++++++ 4 files changed, 68 insertions(+) diff --git a/dlls/ddraw/tests/ddraw1.c b/dlls/ddraw/tests/ddraw1.c index b62d351..a48cf55 100644 --- a/dlls/ddraw/tests/ddraw1.c +++ b/dlls/ddraw/tests/ddraw1.c @@ -3987,6 +3987,22 @@ static void test_primary_palette(void) hr = IDirectDrawSurface_SetPalette(primary, palette); ok(SUCCEEDED(hr), "Failed to set palette, hr %#x.\n", hr); + + /* The Windows 8 testbot attaches the palette to the backbuffer as well, + * and is generally somewhat broken with respect to 8 bpp / palette + * handling. */ + if (SUCCEEDED(IDirectDrawSurface_GetPalette(backbuffer, &tmp))) + { + win_skip("Broken palette handling detected, skipping tests.\n"); + IDirectDrawPalette_Release(tmp); + IDirectDrawPalette_Release(palette); + /* The Windows 8 testbot keeps extra references to the primary and + * backbuffer while in 8 bpp mode. */ + hr = IDirectDraw_RestoreDisplayMode(ddraw); + ok(SUCCEEDED(hr), "Failed to restore display mode, hr %#x.\n", hr); + goto done; + } + refcount = get_refcount((IUnknown *)palette); ok(refcount == 2, "Got unexpected refcount %u.\n", refcount); @@ -4027,6 +4043,7 @@ static void test_primary_palette(void) hr = IDirectDrawSurface_GetPalette(primary, &tmp); ok(hr == DDERR_NOPALETTEATTACHED, "Got unexpected hr %#x.\n", hr); +done: refcount = IDirectDrawSurface_Release(backbuffer); ok(refcount == 1, "Got unexpected refcount %u.\n", refcount); refcount = IDirectDrawSurface_Release(primary); diff --git a/dlls/ddraw/tests/ddraw2.c b/dlls/ddraw/tests/ddraw2.c index ce3be71..cd038c5 100644 --- a/dlls/ddraw/tests/ddraw2.c +++ b/dlls/ddraw/tests/ddraw2.c @@ -5092,6 +5092,22 @@ static void test_primary_palette(void) hr = IDirectDrawSurface_SetPalette(primary, palette); ok(SUCCEEDED(hr), "Failed to set palette, hr %#x.\n", hr); + + /* The Windows 8 testbot attaches the palette to the backbuffer as well, + * and is generally somewhat broken with respect to 8 bpp / palette + * handling. */ + if (SUCCEEDED(IDirectDrawSurface_GetPalette(backbuffer, &tmp))) + { + win_skip("Broken palette handling detected, skipping tests.\n"); + IDirectDrawPalette_Release(tmp); + IDirectDrawPalette_Release(palette); + /* The Windows 8 testbot keeps extra references to the primary and + * backbuffer while in 8 bpp mode. */ + hr = IDirectDraw2_RestoreDisplayMode(ddraw); + ok(SUCCEEDED(hr), "Failed to restore display mode, hr %#x.\n", hr); + goto done; + } + refcount = get_refcount((IUnknown *)palette); ok(refcount == 2, "Got unexpected refcount %u.\n", refcount); @@ -5132,6 +5148,7 @@ static void test_primary_palette(void) hr = IDirectDrawSurface_GetPalette(primary, &tmp); ok(hr == DDERR_NOPALETTEATTACHED, "Got unexpected hr %#x.\n", hr); +done: refcount = IDirectDrawSurface_Release(backbuffer); ok(refcount == 1, "Got unexpected refcount %u.\n", refcount); refcount = IDirectDrawSurface_Release(primary); diff --git a/dlls/ddraw/tests/ddraw4.c b/dlls/ddraw/tests/ddraw4.c index 68f32ee..8becc1f 100644 --- a/dlls/ddraw/tests/ddraw4.c +++ b/dlls/ddraw/tests/ddraw4.c @@ -5752,6 +5752,22 @@ static void test_primary_palette(void) hr = IDirectDrawSurface4_SetPalette(primary, palette); ok(SUCCEEDED(hr), "Failed to set palette, hr %#x.\n", hr); + + /* The Windows 8 testbot attaches the palette to the backbuffer as well, + * and is generally somewhat broken with respect to 8 bpp / palette + * handling. */ + if (SUCCEEDED(IDirectDrawSurface4_GetPalette(backbuffer, &tmp))) + { + win_skip("Broken palette handling detected, skipping tests.\n"); + IDirectDrawPalette_Release(tmp); + IDirectDrawPalette_Release(palette); + /* The Windows 8 testbot keeps extra references to the primary and + * backbuffer while in 8 bpp mode. */ + hr = IDirectDraw4_RestoreDisplayMode(ddraw); + ok(SUCCEEDED(hr), "Failed to restore display mode, hr %#x.\n", hr); + goto done; + } + refcount = get_refcount((IUnknown *)palette); ok(refcount == 2, "Got unexpected refcount %u.\n", refcount); @@ -5792,6 +5808,7 @@ static void test_primary_palette(void) hr = IDirectDrawSurface4_GetPalette(primary, &tmp); ok(hr == DDERR_NOPALETTEATTACHED, "Got unexpected hr %#x.\n", hr); +done: refcount = IDirectDrawSurface4_Release(backbuffer); ok(refcount == 1, "Got unexpected refcount %u.\n", refcount); refcount = IDirectDrawSurface4_Release(primary); diff --git a/dlls/ddraw/tests/ddraw7.c b/dlls/ddraw/tests/ddraw7.c index bed5cb8..375403c 100644 --- a/dlls/ddraw/tests/ddraw7.c +++ b/dlls/ddraw/tests/ddraw7.c @@ -5631,6 +5631,22 @@ static void test_primary_palette(void) hr = IDirectDrawSurface7_SetPalette(primary, palette); ok(SUCCEEDED(hr), "Failed to set palette, hr %#x.\n", hr); + + /* The Windows 8 testbot attaches the palette to the backbuffer as well, + * and is generally somewhat broken with respect to 8 bpp / palette + * handling. */ + if (SUCCEEDED(IDirectDrawSurface7_GetPalette(backbuffer, &tmp))) + { + win_skip("Broken palette handling detected, skipping tests.\n"); + IDirectDrawPalette_Release(tmp); + IDirectDrawPalette_Release(palette); + /* The Windows 8 testbot keeps extra references to the primary and + * backbuffer while in 8 bpp mode. */ + hr = IDirectDraw7_RestoreDisplayMode(ddraw); + ok(SUCCEEDED(hr), "Failed to restore display mode, hr %#x.\n", hr); + goto done; + } + refcount = get_refcount((IUnknown *)palette); ok(refcount == 2, "Got unexpected refcount %u.\n", refcount); @@ -5671,6 +5687,7 @@ static void test_primary_palette(void) hr = IDirectDrawSurface7_GetPalette(primary, &tmp); ok(hr == DDERR_NOPALETTEATTACHED, "Got unexpected hr %#x.\n", hr); +done: refcount = IDirectDrawSurface7_Release(backbuffer); ok(refcount == 1, "Got unexpected refcount %u.\n", refcount); refcount = IDirectDrawSurface7_Release(primary);

1 0

Henri Verbeet : ddraw: Disable lighting when a 0 material handle is set.
by Alexandre Julliard 30 Apr '14

30 Apr '14

Module: wine Branch: master Commit: 91a615ee4dc4efb7ef9929d6422421ccd2f039e4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=91a615ee4dc4efb7ef9929d64… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Apr 29 22:02:46 2014 +0200 ddraw: Disable lighting when a 0 material handle is set. --- dlls/ddraw/device.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/dlls/ddraw/device.c b/dlls/ddraw/device.c index 88142a9..925d97e 100644 --- a/dlls/ddraw/device.c +++ b/dlls/ddraw/device.c @@ -3571,15 +3571,11 @@ static HRESULT WINAPI d3d_device7_DrawPrimitive_FPUPreserve(IDirect3DDevice7 *if static void setup_lighting(const struct d3d_device *device, DWORD fvf, DWORD flags) { - BOOL enable; + BOOL enable = TRUE; /* Ignore the D3DFVF_XYZRHW case here, wined3d takes care of that */ - if (flags & D3DDP_DONOTLIGHT) + if (!device->material || !(fvf & D3DFVF_NORMAL) || (flags & D3DDP_DONOTLIGHT)) enable = FALSE; - else if (!(fvf & D3DFVF_NORMAL)) - enable = FALSE; - else - enable = TRUE; wined3d_device_set_render_state(device->wined3d_device, WINED3D_RS_LIGHTING, enable); }

1 0

Henri Verbeet : ddraw: Reject setting a 0 material handle in d3d_viewport_SetBackground().
by Alexandre Julliard 30 Apr '14

30 Apr '14

Module: wine Branch: master Commit: 4ff220f94b30db5c9b58734270ba9599cb1c7101 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4ff220f94b30db5c9b5873427… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Apr 29 22:02:47 2014 +0200 ddraw: Reject setting a 0 material handle in d3d_viewport_SetBackground(). --- dlls/ddraw/viewport.c | 29 ++++------------------------- 1 file changed, 4 insertions(+), 25 deletions(-) diff --git a/dlls/ddraw/viewport.c b/dlls/ddraw/viewport.c index 99c6ec9..e557161 100644 --- a/dlls/ddraw/viewport.c +++ b/dlls/ddraw/viewport.c @@ -524,39 +524,18 @@ static HRESULT WINAPI d3d_viewport_LightElements(IDirect3DViewport3 *iface, return DDERR_UNSUPPORTED; } -/***************************************************************************** - * IDirect3DViewport3::SetBackground - * - * Sets the background material - * - * Params: - * hMat: Handle from a IDirect3DMaterial interface - * - * Returns: - * D3D_OK on success - * - *****************************************************************************/ -static HRESULT WINAPI d3d_viewport_SetBackground(IDirect3DViewport3 *iface, D3DMATERIALHANDLE hMat) +static HRESULT WINAPI d3d_viewport_SetBackground(IDirect3DViewport3 *iface, D3DMATERIALHANDLE material) { struct d3d_viewport *viewport = impl_from_IDirect3DViewport3(iface); struct d3d_material *m; - TRACE("iface %p, material %#x.\n", iface, hMat); + TRACE("iface %p, material %#x.\n", iface, material); wined3d_mutex_lock(); - if (!hMat) - { - viewport->background = NULL; - TRACE("Setting background to NULL\n"); - wined3d_mutex_unlock(); - return D3D_OK; - } - - m = ddraw_get_object(&viewport->ddraw->d3ddevice->handle_table, hMat - 1, DDRAW_HANDLE_MATERIAL); - if (!m) + if (!(m = ddraw_get_object(&viewport->ddraw->d3ddevice->handle_table, material - 1, DDRAW_HANDLE_MATERIAL))) { - WARN("Invalid material handle.\n"); + WARN("Invalid material handle %#x.\n", material); wined3d_mutex_unlock(); return DDERR_INVALIDPARAMS; }

1 0

Henri Verbeet : ddraw: Only output a warning when clearing a viewport without background material.
by Alexandre Julliard 30 Apr '14

30 Apr '14

Module: wine Branch: master Commit: 27a0b8a5f0a1e385efd451f615d2bba24899ff46 URL: http://source.winehq.org/git/wine.git/?a=commit;h=27a0b8a5f0a1e385efd451f61… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Apr 29 22:02:48 2014 +0200 ddraw: Only output a warning when clearing a viewport without background material. --- dlls/ddraw/viewport.c | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/dlls/ddraw/viewport.c b/dlls/ddraw/viewport.c index e557161..a1126ec 100644 --- a/dlls/ddraw/viewport.c +++ b/dlls/ddraw/viewport.c @@ -665,16 +665,13 @@ static HRESULT WINAPI d3d_viewport_Clear(IDirect3DViewport3 *iface, if (flags & D3DCLEAR_TARGET) { - if (This->background == NULL) { - ERR(" Trying to clear the color buffer without background material!\n"); - } + if (!This->background) + WARN("No background material set.\n"); else - { - color = ((int)((This->background->mat.u.diffuse.u1.r) * 255) << 16) - | ((int) ((This->background->mat.u.diffuse.u2.g) * 255) << 8) - | ((int) ((This->background->mat.u.diffuse.u3.b) * 255) << 0) - | ((int) ((This->background->mat.u.diffuse.u4.a) * 255) << 24); - } + color = D3DRGBA(This->background->mat.u.diffuse.u1.r, + This->background->mat.u.diffuse.u2.g, + This->background->mat.u.diffuse.u3.b, + This->background->mat.u.diffuse.u4.a); } /* Need to temporarily activate viewport to clear it. Previously active one will be restored

1 0

Henri Verbeet : ddraw/tests: Add some basic material tests.
by Alexandre Julliard 30 Apr '14

30 Apr '14

Module: wine Branch: master Commit: 7bdbb8c890e1f2525daf9a55e0034dfd5355380a URL: http://source.winehq.org/git/wine.git/?a=commit;h=7bdbb8c890e1f2525daf9a55e… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Apr 30 12:36:15 2014 +0200 ddraw/tests: Add some basic material tests. --- dlls/ddraw/tests/ddraw1.c | 225 ++++++++++++++++++++++++++++++++++++++++++--- dlls/ddraw/tests/ddraw2.c | 197 ++++++++++++++++++++++++++++++++++----- dlls/ddraw/tests/ddraw4.c | 218 ++++++++++++++++++++++++++++++++++++++++--- dlls/ddraw/tests/ddraw7.c | 44 +++++++++ 4 files changed, 631 insertions(+), 53 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=7bdbb8c890e1f2525daf9…

1 0

Alistair Leslie-Hughes : dpnet: Cleanup IDirectPlay8Address.
by Alexandre Julliard 30 Apr '14

30 Apr '14

Module: wine Branch: master Commit: 91414e8ef8340a5afafb3418d6024c100e7dfb8e URL: http://source.winehq.org/git/wine.git/?a=commit;h=91414e8ef8340a5afafb3418d… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Fri Apr 11 09:55:59 2014 +1000 dpnet: Cleanup IDirectPlay8Address. --- dlls/dpnet/address.c | 45 ++++++++++++++++++++++++++------------------- 1 file changed, 26 insertions(+), 19 deletions(-) diff --git a/dlls/dpnet/address.c b/dlls/dpnet/address.c index 796e8a4..9266fea 100644 --- a/dlls/dpnet/address.c +++ b/dlls/dpnet/address.c @@ -58,24 +58,24 @@ static HRESULT WINAPI IDirectPlay8AddressImpl_QueryInterface(IDirectPlay8Address static ULONG WINAPI IDirectPlay8AddressImpl_AddRef(IDirectPlay8Address *iface) { IDirectPlay8AddressImpl *This = impl_from_IDirectPlay8Address(iface); - ULONG refCount = InterlockedIncrement(&This->ref); + ULONG ref = InterlockedIncrement(&This->ref); - TRACE("(%p)->(ref before=%u)\n", This, refCount - 1); + TRACE("(%p) ref=%u\n", This, ref); - return refCount; + return ref; } static ULONG WINAPI IDirectPlay8AddressImpl_Release(IDirectPlay8Address *iface) { IDirectPlay8AddressImpl *This = impl_from_IDirectPlay8Address(iface); - ULONG refCount = InterlockedDecrement(&This->ref); + ULONG ref = InterlockedDecrement(&This->ref); - TRACE("(%p)->(ref before=%u)\n", This, refCount + 1); + TRACE("(%p) ref=%u\n", This, ref); - if (!refCount) { + if (!ref) { HeapFree(GetProcessHeap(), 0, This); } - return refCount; + return ref; } /* returns name of given GUID */ @@ -312,17 +312,24 @@ static const IDirectPlay8AddressVtbl DirectPlay8Address_Vtbl = IDirectPlay8AddressImpl_BuildFromDirectPlay4Address }; -HRESULT DPNET_CreateDirectPlay8Address(LPCLASSFACTORY iface, LPUNKNOWN punkOuter, REFIID riid, LPVOID *ppobj) { - IDirectPlay8AddressImpl* client; +HRESULT DPNET_CreateDirectPlay8Address(IClassFactory *iface, IUnknown *pUnkOuter, REFIID riid, LPVOID *ppobj) +{ + IDirectPlay8AddressImpl* client; + HRESULT ret; - TRACE("(%p, %s, %p)\n", punkOuter, debugstr_guid(riid), ppobj); - - client = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(IDirectPlay8AddressImpl)); - if (NULL == client) { - *ppobj = NULL; - return E_OUTOFMEMORY; - } - client->IDirectPlay8Address_iface.lpVtbl = &DirectPlay8Address_Vtbl; - client->ref = 0; /* will be inited with QueryInterface */ - return IDirectPlay8AddressImpl_QueryInterface (&client->IDirectPlay8Address_iface, riid, ppobj); + TRACE("(%p, %s, %p)\n", pUnkOuter, debugstr_guid(riid), ppobj); + + *ppobj = NULL; + + client = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(IDirectPlay8AddressImpl)); + if (!client) + return E_OUTOFMEMORY; + + client->IDirectPlay8Address_iface.lpVtbl = &DirectPlay8Address_Vtbl; + client->ref = 1; + + ret = IDirectPlay8AddressImpl_QueryInterface(&client->IDirectPlay8Address_iface, riid, ppobj); + IDirectPlay8AddressImpl_Release(&client->IDirectPlay8Address_iface); + + return ret; }

1 0

Henri Verbeet : ddraw: Cleanup the D3DOP_PROCESSVERTICES implementation a bit.
by Alexandre Julliard 29 Apr '14

29 Apr '14

Module: wine Branch: master Commit: b2e2b42b73e7e17acab7cc737874d2c6aa92b701 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b2e2b42b73e7e17acab7cc737… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Apr 29 10:41:04 2014 +0200 ddraw: Cleanup the D3DOP_PROCESSVERTICES implementation a bit. --- dlls/ddraw/executebuffer.c | 227 +++++++++++++++++--------------------------- 1 file changed, 88 insertions(+), 139 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=b2e2b42b73e7e17acab7c…

1 0