July 2016 - Wine-Commits - list.winehq.org

Michael Stefaniuc : shell32/tests: Avoid casts from a COM object to an iface.
by Alexandre Julliard July 3, 2016

July 3, 2016

Module: wine Branch: master Commit: df63e1a169733431d7e83bebdc5f47e36007e3f3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=df63e1a169733431d7e83bebd… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Jun 29 09:53:10 2016 +0200 shell32/tests: Avoid casts from a COM object to an iface. Signed-off-by: Michael Stefaniuc <mstefani(a)redhat.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/shell32/tests/shlfolder.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/shell32/tests/shlfolder.c b/dlls/shell32/tests/shlfolder.c index e80e8a4..1591b7b 100644 --- a/dlls/shell32/tests/shlfolder.c +++ b/dlls/shell32/tests/shlfolder.c @@ -3088,7 +3088,7 @@ static void test_SHGetIDListFromObject(void) punkimpl->ifaces = ifaces; punkimpl->unknown = 0; - hres = pSHGetIDListFromObject((IUnknown*)punkimpl, &pidl); + hres = pSHGetIDListFromObject(&punkimpl->IUnknown_iface, &pidl); ok(hres == E_NOINTERFACE, "Got %x\n", hres); ok(ifaces[0].count, "interface not requested.\n"); ok(ifaces[1].count, "interface not requested.\n"); @@ -3260,7 +3260,7 @@ static void test_SHGetItemFromObject(void) punkimpl->unknown = 0; /* The same as SHGetIDListFromObject */ - hres = pSHGetIDListFromObject((IUnknown*)punkimpl, &pidl); + hres = pSHGetIDListFromObject(&punkimpl->IUnknown_iface, &pidl); ok(hres == E_NOINTERFACE, "Got %x\n", hres); ok(ifaces[0].count, "interface not requested.\n"); ok(ifaces[1].count, "interface not requested.\n");

1 0

Vincent Povirk : gdiplus: Clip metafile drawing to destination points.
by Alexandre Julliard July 3, 2016

July 3, 2016

Module: wine Branch: master Commit: 10516308a8d018e6016431d069f38b9ad292afd3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=10516308a8d018e6016431d06… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Fri Jun 24 12:51:40 2016 -0500 gdiplus: Clip metafile drawing to destination points. Signed-off-by: Vincent Povirk <vincent(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdiplus/gdiplus_private.h | 1 + dlls/gdiplus/metafile.c | 46 ++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 47 insertions(+) diff --git a/dlls/gdiplus/gdiplus_private.h b/dlls/gdiplus/gdiplus_private.h index 22478a3..8924ae6 100644 --- a/dlls/gdiplus/gdiplus_private.h +++ b/dlls/gdiplus/gdiplus_private.h @@ -358,6 +358,7 @@ struct GpMetafile{ GpMatrix *world_transform; GpUnit page_unit; REAL page_scale; + GpRegion *base_clip; /* clip region in device space for all metafile output */ }; struct GpBitmap{ diff --git a/dlls/gdiplus/metafile.c b/dlls/gdiplus/metafile.c index ee16d8e..ca8aac4 100644 --- a/dlls/gdiplus/metafile.c +++ b/dlls/gdiplus/metafile.c @@ -591,6 +591,11 @@ static void METAFILE_PlaybackReleaseDC(GpMetafile *metafile) } } +static GpStatus METAFILE_PlaybackUpdateClip(GpMetafile *metafile) +{ + return GdipCombineRegionRegion(metafile->playback_graphics->clip, metafile->base_clip, CombineModeReplace); +} + static GpStatus METAFILE_PlaybackUpdateWorldTransform(GpMetafile *metafile) { GpMatrix *real_transform; @@ -822,6 +827,7 @@ GpStatus WINGDIPAPI GdipEnumerateMetafileSrcRectDestPoints(GpGraphics *graphics, GpStatus stat; GpMetafile *real_metafile = (GpMetafile*)metafile; /* whoever made this const was joking */ GraphicsContainer state; + GpPath *dst_path; TRACE("(%p,%p,%p,%i,%p,%i,%p,%p,%p)\n", graphics, metafile, destPoints, count, srcRect, srcUnit, callback, callbackData, @@ -862,6 +868,38 @@ GpStatus WINGDIPAPI GdipEnumerateMetafileSrcRectDestPoints(GpGraphics *graphics, stat = GdipSetPageUnit(graphics, UnitPixel); if (stat == Ok) + stat = GdipResetWorldTransform(graphics); + + if (stat == Ok) + stat = GdipCreateRegion(&real_metafile->base_clip); + + if (stat == Ok) + stat = GdipGetClip(graphics, real_metafile->base_clip); + + if (stat == Ok) + stat = GdipCreatePath(FillModeAlternate, &dst_path); + + if (stat == Ok) + { + GpPointF clip_points[4]; + + clip_points[0] = real_metafile->playback_points[0]; + clip_points[1] = real_metafile->playback_points[1]; + clip_points[2].X = real_metafile->playback_points[1].X + real_metafile->playback_points[2].X + - real_metafile->playback_points[0].X; + clip_points[2].Y = real_metafile->playback_points[1].Y + real_metafile->playback_points[2].Y + - real_metafile->playback_points[0].Y; + clip_points[3] = real_metafile->playback_points[2]; + + stat = GdipAddPathPolygon(dst_path, clip_points, 4); + + if (stat == Ok) + stat = GdipCombineRegionPath(real_metafile->base_clip, dst_path, CombineModeIntersect); + + GdipDeletePath(dst_path); + } + + if (stat == Ok) stat = GdipCreateMatrix(&real_metafile->world_transform); if (stat == Ok) @@ -871,6 +909,11 @@ GpStatus WINGDIPAPI GdipEnumerateMetafileSrcRectDestPoints(GpGraphics *graphics, stat = METAFILE_PlaybackUpdateWorldTransform(real_metafile); } + if (stat == Ok) + { + stat = METAFILE_PlaybackUpdateClip(real_metafile); + } + if (stat == Ok && (metafile->metafile_type == MetafileTypeEmf || metafile->metafile_type == MetafileTypeWmfPlaceable || metafile->metafile_type == MetafileTypeWmf)) @@ -884,6 +927,9 @@ GpStatus WINGDIPAPI GdipEnumerateMetafileSrcRectDestPoints(GpGraphics *graphics, GdipDeleteMatrix(real_metafile->world_transform); real_metafile->world_transform = NULL; + GdipDeleteRegion(real_metafile->base_clip); + real_metafile->base_clip = NULL; + GdipEndContainer(graphics, state); }

1 0

Aaryaman Vasishta : d3drm: Rearrange version 1 functions after version 2 for IDirect3DRMViewport*.
by Alexandre Julliard July 3, 2016

July 3, 2016

Module: wine Branch: master Commit: 49f37796d5a50ed8d261d19c6f35d1ed8271fd81 URL: http://source.winehq.org/git/wine.git/?a=commit;h=49f37796d5a50ed8d261d19c6… Author: Aaryaman Vasishta <jem456.vasishta(a)gmail.com> Date: Wed Jun 29 15:39:03 2016 +0200 d3drm: Rearrange version 1 functions after version 2 for IDirect3DRMViewport*. Signed-off-by: Aaryaman Vasishta <jem456.vasishta(a)gmail.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3drm/viewport.c | 688 +++++++++++++++++++++++++------------------------- 1 file changed, 344 insertions(+), 344 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=49f37796d5a50ed8d261d…

1 0

Aaryaman Vasishta : d3drm: Introduce d3drm_viewport_destroy().
by Alexandre Julliard July 3, 2016

July 3, 2016

Module: wine Branch: master Commit: be7bca55e3799926b4002f343a8edca61d8564d1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=be7bca55e3799926b4002f343… Author: Aaryaman Vasishta <jem456.vasishta(a)gmail.com> Date: Wed Jun 29 15:39:02 2016 +0200 d3drm: Introduce d3drm_viewport_destroy(). Signed-off-by: Aaryaman Vasishta <jem456.vasishta(a)gmail.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3drm/viewport.c | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/dlls/d3drm/viewport.c b/dlls/d3drm/viewport.c index ab9ccde..434d5e9 100644 --- a/dlls/d3drm/viewport.c +++ b/dlls/d3drm/viewport.c @@ -39,6 +39,14 @@ static inline struct d3drm_viewport *impl_from_IDirect3DRMViewport2(IDirect3DRMV return CONTAINING_RECORD(iface, struct d3drm_viewport, IDirect3DRMViewport2_iface); } +static void d3drm_viewport_destroy(struct d3drm_viewport *viewport) +{ + TRACE("viewport %p.\n", viewport); + + d3drm_object_cleanup((IDirect3DRMObject *)&viewport->IDirect3DRMViewport_iface, &viewport->obj); + HeapFree(GetProcessHeap(), 0, viewport); +} + static HRESULT WINAPI d3drm_viewport1_QueryInterface(IDirect3DRMViewport *iface, REFIID riid, void **out) { struct d3drm_viewport *viewport = impl_from_IDirect3DRMViewport(iface); @@ -84,10 +92,7 @@ static ULONG WINAPI d3drm_viewport1_Release(IDirect3DRMViewport *iface) TRACE("%p decreasing refcount to %u.\n", iface, refcount); if (!refcount) - { - d3drm_object_cleanup((IDirect3DRMObject*)iface, &viewport->obj); - HeapFree(GetProcessHeap(), 0, viewport); - } + d3drm_viewport_destroy(viewport); return refcount; }

1 0

Michael Stefaniuc : mciavi32: Use SetRect() instead of open coding it.
by Alexandre Julliard July 3, 2016

July 3, 2016

Module: wine Branch: master Commit: 9fd4429d493ff4483c361a466a8cb21206d60b08 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9fd4429d493ff4483c361a466… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Jun 29 09:54:30 2016 +0200 mciavi32: Use SetRect() instead of open coding it. Signed-off-by: Michael Stefaniuc <mstefani(a)redhat.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mciavi32/mmoutput.c | 6 +----- dlls/mciavi32/wnd.c | 24 +++++++++--------------- 2 files changed, 10 insertions(+), 20 deletions(-) diff --git a/dlls/mciavi32/mmoutput.c b/dlls/mciavi32/mmoutput.c index c932ab4..ee0481c 100644 --- a/dlls/mciavi32/mmoutput.c +++ b/dlls/mciavi32/mmoutput.c @@ -140,11 +140,7 @@ static BOOL MCIAVI_GetInfoVideo(WINE_MCIAVI* wma, const MMCKINFO* mmckList, MMCK TRACE("bih.biClrUsed=%d\n", wma->inbih->biClrUsed); TRACE("bih.biClrImportant=%d\n", wma->inbih->biClrImportant); - wma->source.left = 0; - wma->source.top = 0; - wma->source.right = wma->inbih->biWidth; - wma->source.bottom = wma->inbih->biHeight; - + SetRect(&wma->source, 0, 0, wma->inbih->biWidth, wma->inbih->biHeight); wma->dest = wma->source; return TRUE; diff --git a/dlls/mciavi32/wnd.c b/dlls/mciavi32/wnd.c index c7dcf61..66c7565 100644 --- a/dlls/mciavi32/wnd.c +++ b/dlls/mciavi32/wnd.c @@ -122,9 +122,11 @@ BOOL MCIAVI_CreateWindow(WINE_MCIAVI* wma, DWORD dwFlags, LPMCI_DGV_OPEN_PARM if (dwFlags & MCI_DGV_OPEN_PARENT) hParent = lpOpenParms->hWndParent; if (dwFlags & MCI_DGV_OPEN_WS) dwStyle = lpOpenParms->dwStyle; - rc.left = rc.top = 0; - rc.right = (wma->hic ? wma->outbih : wma->inbih)->biWidth; - rc.bottom = (wma->hic ? wma->outbih : wma->inbih)->biHeight; + if (wma->hic) + SetRect(&rc, 0, 0, wma->outbih->biWidth, wma->outbih->biHeight); + else + SetRect(&rc, 0, 0, wma->inbih->biWidth, wma->inbih->biHeight); + AdjustWindowRect(&rc, dwStyle, FALSE); if (!(dwStyle & (WS_CHILD|WS_POPUP))) /* overlapped window ? */ { @@ -161,10 +163,8 @@ DWORD MCIAVI_mciPut(UINT wDevID, DWORD dwFlags, LPMCI_DGV_PUT_PARMS lpParms) if (dwFlags & MCI_DGV_RECT) { /* In MCI, RECT structure is used differently: rc.right = width & rc.bottom = height * So convert input MCI RECT into a normal RECT */ - rc.left = lpParms->rc.left; - rc.top = lpParms->rc.top; - rc.right = lpParms->rc.left + lpParms->rc.right; - rc.bottom = lpParms->rc.top + lpParms->rc.bottom; + SetRect(&rc, lpParms->rc.left, lpParms->rc.top, lpParms->rc.left + lpParms->rc.right, + lpParms->rc.top + lpParms->rc.bottom); } else { GetClientRect(wma->hWndPaint, &rc); } @@ -235,10 +235,7 @@ DWORD MCIAVI_mciWhere(UINT wDevID, DWORD dwFlags, LPMCI_DGV_RECT_PARMS lpParms) } if (dwFlags & MCI_DGV_WHERE_SOURCE) { if (dwFlags & MCI_DGV_WHERE_MAX) { - rc.left = 0; - rc.top = 0; - rc.right = wma->inbih->biWidth; - rc.bottom = wma->inbih->biHeight; + SetRect(&rc, 0, 0, wma->inbih->biWidth, wma->inbih->biHeight); TRACE("WHERE_SOURCE_MAX %s\n", wine_dbgstr_rect(&rc)); } else { TRACE("WHERE_SOURCE %s\n", wine_dbgstr_rect(&wma->source)); @@ -265,10 +262,7 @@ DWORD MCIAVI_mciWhere(UINT wDevID, DWORD dwFlags, LPMCI_DGV_RECT_PARMS lpParms) /* In MCI, RECT structure is used differently: rc.right = width & rc.bottom = height * So convert the normal RECT into a MCI RECT before returning */ - lpParms->rc.left = rc.left; - lpParms->rc.top = rc.top; - lpParms->rc.right = rc.right - rc.left; - lpParms->rc.bottom = rc.bottom - rc.top; + SetRect(&lpParms->rc, rc.left, rc.top, rc.right - rc.left, rc.bottom - rc.top); LeaveCriticalSection(&wma->cs); return 0;

1 0

Benjamin Hodgetts : wined3d: Add Intel Iris chips to identification list.
by Alexandre Julliard July 3, 2016

July 3, 2016

Module: wine Branch: master Commit: a14e5fb1bb19a2bd7cf96e59b0fb4ee70f304401 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a14e5fb1bb19a2bd7cf96e59b… Author: Benjamin Hodgetts <ben(a)xnode.org> Date: Tue Jun 28 23:59:26 2016 +0100 wined3d: Add Intel Iris chips to identification list. Add the newer and high-end Intel Iris GPU chips to the current identification list. Signed-off-by: Benjamin Hodgetts <ben(a)xnode.org> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/directx.c | 7 +++++++ dlls/wined3d/wined3d_private.h | 3 +++ 2 files changed, 10 insertions(+) diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c index 03008cc..7d80752 100644 --- a/dlls/wined3d/directx.c +++ b/dlls/wined3d/directx.c @@ -1393,10 +1393,13 @@ static const struct gpu_description gpu_description_table[] = {HW_VENDOR_INTEL, CARD_INTEL_IVBS, "Intel(R) HD Graphics Family", DRIVER_INTEL_HD4000, 1536}, {HW_VENDOR_INTEL, CARD_INTEL_HWD, "Intel(R) HD Graphics 4600", DRIVER_INTEL_HD4000, 1536}, {HW_VENDOR_INTEL, CARD_INTEL_HWM, "Intel(R) HD Graphics 4600", DRIVER_INTEL_HD4000, 1536}, + {HW_VENDOR_INTEL, CARD_INTEL_IG6100, "Intel(R) Iris(TM) Graphics 6100", DRIVER_INTEL_HD4000, 2048}, + {HW_VENDOR_INTEL, CARD_INTEL_IP6200, "Intel(R) Iris(TM) Pro Graphics 6200", DRIVER_INTEL_HD4000, 2048}, {HW_VENDOR_INTEL, CARD_INTEL_HD520, "Intel(R) HD Graphics 520", DRIVER_INTEL_HD4000, 2048}, {HW_VENDOR_INTEL, CARD_INTEL_HD530_1, "Intel(R) HD Graphics 530", DRIVER_INTEL_HD4000, 2048}, {HW_VENDOR_INTEL, CARD_INTEL_HD530_2, "Intel(R) HD Graphics 530", DRIVER_INTEL_HD4000, 2048}, {HW_VENDOR_INTEL, CARD_INTEL_HD540, "Intel(R) Iris(TM) Graphics 540", DRIVER_INTEL_HD4000, 2048}, + {HW_VENDOR_INTEL, CARD_INTEL_IPP580, "Intel(R) Iris(TM) Pro Graphics P580", DRIVER_INTEL_HD4000, 2048}, }; static const struct driver_version_information *get_driver_version_info(enum wined3d_display_driver driver, @@ -2003,7 +2006,11 @@ cards_amd_binary[] = cards_intel[] = { /* Skylake */ + {"Iris Pro Graphics P580", CARD_INTEL_IPP580}, {"Skylake", CARD_INTEL_HD520}, + /* Broadwell */ + {"Iris Pro 6200", CARD_INTEL_IP6200}, + {"Iris 6100", CARD_INTEL_IG6100}, /* Haswell */ {"Haswell Mobile", CARD_INTEL_HWM}, /* Ivybridge */ diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index b82ec3d..184d7b5 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1936,10 +1936,13 @@ enum wined3d_pci_device CARD_INTEL_IVBS = 0x015a, CARD_INTEL_HWD = 0x0412, CARD_INTEL_HWM = 0x0416, + CARD_INTEL_IG6100 = 0x162b, + CARD_INTEL_IP6200 = 0x1622, CARD_INTEL_HD520 = 0x1916, CARD_INTEL_HD530_1 = 0x1912, CARD_INTEL_HD530_2 = 0x191b, CARD_INTEL_HD540 = 0x1926, + CARD_INTEL_IPP580 = 0x193d, }; struct wined3d_fbo_ops

1 0

Michael Stefaniuc : d3d9/tests: Use SetRect() instead of open coding it.
by Alexandre Julliard July 3, 2016

July 3, 2016

Module: wine Branch: master Commit: 080c74497de63a237c6d20799f83dd33d092b6f0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=080c74497de63a237c6d20799… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Jun 27 09:57:49 2016 +0200 d3d9/tests: Use SetRect() instead of open coding it. Signed-off-by: Michael Stefaniuc <mstefani(a)redhat.de> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d9/tests/device.c | 14 ++++---------- dlls/d3d9/tests/visual.c | 5 +---- 2 files changed, 5 insertions(+), 14 deletions(-) diff --git a/dlls/d3d9/tests/device.c b/dlls/d3d9/tests/device.c index e73af1f..715e34c 100644 --- a/dlls/d3d9/tests/device.c +++ b/dlls/d3d9/tests/device.c @@ -1990,10 +1990,7 @@ static void test_reset(void) ok(d3dpp.BackBufferHeight == 300, "Got unexpected BackBufferHeight %u.\n", d3dpp.BackBufferHeight); IDirect3DSwapChain9_Release(swapchain); - winrect.left = 0; - winrect.top = 0; - winrect.right = 200; - winrect.bottom = 150; + SetRect(&winrect, 0, 0, 200, 150); ok(AdjustWindowRect(&winrect, WS_OVERLAPPEDWINDOW, FALSE), "AdjustWindowRect failed\n"); ok(SetWindowPos(hwnd, NULL, 0, 0, winrect.right-winrect.left, @@ -8465,17 +8462,14 @@ static void test_surface_blocks(void) hr = IDirect3DTexture9_UnlockRect(texture, 1); ok(SUCCEEDED(hr), "Failed lock texture, hr %#x.\n", hr); - rect.left = 0; - rect.top = 0; - rect.right = formats[i].block_width == 1 ? 1 : formats[i].block_width >> 1; - rect.bottom = formats[i].block_height == 1 ? 1 : formats[i].block_height >> 1; + SetRect(&rect, 0, 0, formats[i].block_width == 1 ? 1 : formats[i].block_width >> 1, + formats[i].block_height == 1 ? 1 : formats[i].block_height >> 1); hr = IDirect3DTexture9_LockRect(texture, 1, &locked_rect, &rect, 0); ok(SUCCEEDED(hr), "Failed lock texture, hr %#x.\n", hr); hr = IDirect3DTexture9_UnlockRect(texture, 1); ok(SUCCEEDED(hr), "Failed lock texture, hr %#x.\n", hr); - rect.right = formats[i].block_width; - rect.bottom = formats[i].block_height; + SetRect(&rect, 0, 0, formats[i].block_width, formats[i].block_height); hr = IDirect3DTexture9_LockRect(texture, 1, &locked_rect, &rect, 0); ok(hr == D3DERR_INVALIDCALL, "Got unexpected hr %#x.\n", hr); if (SUCCEEDED(hr)) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index a308311..a2adc54 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -1041,10 +1041,7 @@ static void clear_test(void) hr = IDirect3DDevice9_Clear(device, 0, NULL, D3DCLEAR_TARGET, 0xffffffff, 0.0, 0); ok(hr == D3D_OK, "IDirect3DDevice9_Clear failed with %08x\n", hr); - scissor.left = 160; - scissor.right = 480; - scissor.top = 120; - scissor.bottom = 360; + SetRect(&scissor, 160, 120, 480, 360); hr = IDirect3DDevice9_SetScissorRect(device, &scissor); ok(hr == D3D_OK, "IDirect3DDevice_SetScissorRect failed with %08x\n", hr); hr = IDirect3DDevice9_SetRenderState(device, D3DRS_SCISSORTESTENABLE, TRUE);

1 0

Michael Stefaniuc : cards: Use SetRect() instead of open coding it.
by Alexandre Julliard July 3, 2016

July 3, 2016

Module: wine Branch: master Commit: 88fb88508804ed3c8e815e8f59a6c94e75c489b5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=88fb88508804ed3c8e815e8f5… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Jun 27 09:45:31 2016 +0200 cards: Use SetRect() instead of open coding it. Signed-off-by: Michael Stefaniuc <mstefani(a)redhat.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/cards/cards.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/dlls/cards/cards.c b/dlls/cards/cards.c index b9506ea..9bec966 100644 --- a/dlls/cards/cards.c +++ b/dlls/cards/cards.c @@ -151,10 +151,7 @@ BOOL WINAPI cdtDrawExt(HDC hdc, int x, int y, int dx, int dy, int card, int mode HBRUSH hBrush; RECT rect; hBrush = CreateSolidBrush(color); - rect.left = x; - rect.top = y; - rect.right = x + cardWidth - 1; - rect.bottom = y + cardHeight - 1; + SetRect(&rect, x, y, x + cardWidth - 1, y + cardHeight - 1); FillRect(hdc, &rect, hBrush); }

1 0

Michael Stefaniuc : d3dx9/tests: Use SetRect() instead of coding it.
by Alexandre Julliard July 3, 2016

July 3, 2016

Module: wine Branch: master Commit: 30ee087d065b8f4d68b08be6ba847699131f5494 URL: http://source.winehq.org/git/wine.git/?a=commit;h=30ee087d065b8f4d68b08be6b… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Tue Jun 28 10:32:31 2016 +0200 d3dx9/tests: Use SetRect() instead of coding it. Signed-off-by: Michael Stefaniuc <mstefani(a)redhat.de> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3dx9_36/sprite.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/dlls/d3dx9_36/sprite.c b/dlls/d3dx9_36/sprite.c index 4a7437f..17ad16a 100644 --- a/dlls/d3dx9_36/sprite.c +++ b/dlls/d3dx9_36/sprite.c @@ -377,12 +377,10 @@ static HRESULT WINAPI d3dx9_sprite_Draw(ID3DXSprite *iface, IDirect3DTexture9 *t This->sprites[This->sprite_count].texw=texdesc.Width; This->sprites[This->sprite_count].texh=texdesc.Height; - if(rect==NULL) { - This->sprites[This->sprite_count].rect.left=0; - This->sprites[This->sprite_count].rect.top=0; - This->sprites[This->sprite_count].rect.right=texdesc.Width; - This->sprites[This->sprite_count].rect.bottom=texdesc.Height; - } else This->sprites[This->sprite_count].rect=*rect; + if (rect) + This->sprites[This->sprite_count].rect = *rect; + else + SetRect(&This->sprites[This->sprite_count].rect, 0, 0, texdesc.Width, texdesc.Height); if(center==NULL) { This->sprites[This->sprite_count].center.x=0.0f;

1 0

Michael Stefaniuc : d3dx10/tests: Use SetRect() instead of open coding it.
by Alexandre Julliard July 3, 2016

July 3, 2016

Module: wine Branch: master Commit: 6329f37d5c0b3b6f2ffb5a4eeaf18fe18bbdafe6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6329f37d5c0b3b6f2ffb5a4ee… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Tue Jun 28 10:24:43 2016 +0200 d3dx10/tests: Use SetRect() instead of open coding it. Signed-off-by: Michael Stefaniuc <mstefani(a)redhat.de> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3dx10_43/tests/d3dx10.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/dlls/d3dx10_43/tests/d3dx10.c b/dlls/d3dx10_43/tests/d3dx10.c index 44c2333..423ab4d 100644 --- a/dlls/d3dx10_43/tests/d3dx10.c +++ b/dlls/d3dx10_43/tests/d3dx10.c @@ -344,10 +344,7 @@ float4 main(float4 color : COLOR) : SV_TARGET for (i = 0; i < D3D10_VIEWPORT_AND_SCISSORRECT_OBJECT_COUNT_PER_PIPELINE; ++i) { - tmp_rect[i].left = i; - tmp_rect[i].top = i * 2; - tmp_rect[i].right = i + 1; - tmp_rect[i].bottom = (i + 1) * 2; + SetRect(&tmp_rect[i], i, i * 2, i + 1, (i + 1) * 2); tmp_viewport[i].TopLeftX = i * 3; tmp_viewport[i].TopLeftY = i * 4;

1 0