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

Daniel Lehman : msvcp90: Add codecvt<wchar_t>::do_encoding that returns 0.
by Alexandre Julliard April 8, 2016

April 8, 2016

Module: wine Branch: master Commit: bd92393a5fbe7b65119cffa98144e5e60ac32eb6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bd92393a5fbe7b65119cffa98… Author: Daniel Lehman <dlehman(a)esri.com> Date: Mon Apr 4 21:08:12 2016 -0700 msvcp90: Add codecvt<wchar_t>::do_encoding that returns 0. Signed-off-by: Daniel Lehman <dlehman(a)esri.com> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msvcp110/msvcp110.spec | 12 ++++++------ dlls/msvcp120/msvcp120.spec | 12 ++++++------ dlls/msvcp120_app/msvcp120_app.spec | 12 ++++++------ dlls/msvcp60/msvcp60.spec | 4 ++-- dlls/msvcp70/msvcp70.spec | 8 ++++---- dlls/msvcp71/msvcp71.spec | 8 ++++---- dlls/msvcp80/msvcp80.spec | 8 ++++---- dlls/msvcp90/locale.c | 19 +++++++++++++++++++ 8 files changed, 51 insertions(+), 32 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=bd92393a5fbe7b65119cf…

1 0

Henri Verbeet : wined3d: Use the texture dimension helpers in surface_blt_special().
by Alexandre Julliard April 8, 2016

April 8, 2016

Module: wine Branch: master Commit: 8c3b0aead55654a4b349ca0fc263907252e32bbb URL: http://source.winehq.org/git/wine.git/?a=commit;h=8c3b0aead55654a4b349ca0fc… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Apr 7 19:37:59 2016 +0200 wined3d: Use the texture dimension helpers in surface_blt_special(). Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/surface.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 3e45599..5fd238e 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -2662,6 +2662,7 @@ static HRESULT surface_blt_special(struct wined3d_surface *dst_surface, const RE if ((src_swapchain || src_surface == rt) && !dst_swapchain) { + unsigned int src_width, src_height; /* Blit from render target to texture */ BOOL stretchx; @@ -2695,8 +2696,10 @@ static HRESULT surface_blt_special(struct wined3d_surface *dst_surface, const RE * back buffer. This is slower than reading line per line, thus not used for flipping * -> If the app wants a scaled image with a dest rect that is bigger than the fb, it has to be copied * pixel by pixel. */ - if (!stretchx || dst_rect->right - dst_rect->left > src_surface->resource.width - || dst_rect->bottom - dst_rect->top > src_surface->resource.height) + src_width = wined3d_texture_get_level_width(src_texture, src_surface->texture_level); + src_height = wined3d_texture_get_level_height(src_texture, src_surface->texture_level); + if (!stretchx || dst_rect->right - dst_rect->left > src_width + || dst_rect->bottom - dst_rect->top > src_height) { TRACE("No stretching in x direction, using direct framebuffer -> texture copy.\n"); fb_copy_to_texture_direct(dst_surface, src_surface, src_rect, dst_rect, filter);

1 0

Henri Verbeet : wined3d: Use the texture dimension helpers in surface_translate_drawable_coords().
by Alexandre Julliard April 8, 2016

April 8, 2016

Module: wine Branch: master Commit: 35fdf09643e5f45b7cc688f2c9ceb9b9ade98b6b URL: http://source.winehq.org/git/wine.git/?a=commit;h=35fdf09643e5f45b7cc688f2c… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Apr 7 19:37:58 2016 +0200 wined3d: Use the texture dimension helpers in surface_translate_drawable_coords(). Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/surface.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 301aa16..3e45599 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -2447,9 +2447,10 @@ static void fb_copy_to_texture_hwstretch(struct wined3d_surface *dst_surface, st * origin, while D3D has a top-left origin. */ void surface_translate_drawable_coords(const struct wined3d_surface *surface, HWND window, RECT *rect) { + struct wined3d_texture *texture = surface->container; UINT drawable_height; - if (surface->container->swapchain && surface->container == surface->container->swapchain->front_buffer) + if (texture->swapchain && texture == texture->swapchain->front_buffer) { POINT offset = {0, 0}; RECT windowsize; @@ -2462,7 +2463,7 @@ void surface_translate_drawable_coords(const struct wined3d_surface *surface, HW } else { - drawable_height = surface->resource.height; + drawable_height = wined3d_texture_get_level_height(texture, surface->texture_level); } rect->top = drawable_height - rect->top;

1 0

Nikolay Sivov : user32/tests: Message filtering tests for broadcast messages.
by Alexandre Julliard April 8, 2016

April 8, 2016

Module: wine Branch: master Commit: d154c8424df1e7ca6b585eca14fff3578c82cc59 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d154c8424df1e7ca6b585eca1… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Apr 7 17:46:17 2016 +0300 user32/tests: Message filtering tests for broadcast messages. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/tests/msg.c | 122 ++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 122 insertions(+) diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index 97bf443..3c5914e 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -13991,6 +13991,127 @@ static void test_PostMessage(void) flush_events(); } +static LPARAM g_broadcast_lparam; +static LRESULT WINAPI broadcast_test_proc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam) +{ + WNDPROC oldproc = (WNDPROC)GetWindowLongPtrA(hwnd, GWLP_USERDATA); + + if (wParam == 0xbaadbeef) + g_broadcast_lparam = wParam; + else + g_broadcast_lparam = 0; + + return CallWindowProcA(oldproc, hwnd, message, wParam, lParam); +} + +static void test_broadcast(void) +{ + static const UINT messages[] = + { + WM_USER-1, + WM_USER, + WM_USER+1, + 0xc000-1, + 0xc000, /* lowest possible atom returned by RegisterWindowMessage */ + 0xffff, + }; + WNDPROC oldproc; + unsigned int i; + HWND hwnd; + + hwnd = CreateWindowExA(0, "static", NULL, WS_POPUP, 0, 0, 0, 0, 0, 0, 0, NULL); + ok(hwnd != NULL, "got %p\n", hwnd); + + oldproc = (WNDPROC)SetWindowLongPtrA(hwnd, GWLP_WNDPROC, (LONG_PTR)broadcast_test_proc); + SetWindowLongPtrA(hwnd, GWLP_USERDATA, (LONG_PTR)oldproc); + + for (i = 0; i < sizeof(messages)/sizeof(messages[0]); i++) + { + BOOL ret; + MSG msg; + + flush_events(); + while (PeekMessageA(&msg, 0, 0, 0, PM_REMOVE)) + ; + + /* post, broadcast */ + ret = PostMessageA(HWND_BROADCAST, messages[i], 0, 0); + ok(ret, "%d: got %d, error %d\n", i, ret, GetLastError()); + + memset(&msg, 0xab, sizeof(msg)); + ret = PeekMessageA(&msg, 0, 0, 0, PM_REMOVE); + if (messages[i] < WM_USER || messages[i] >= 0xc000) + { + ok(ret, "%d: message %04x, got %d, error %d\n", i, messages[i], ret, GetLastError()); + ok(msg.hwnd == hwnd, "%d: got %p\n", i, msg.hwnd); + } + else + { + todo_wine + ok(!ret, "%d: message %04x, got %d, error %d\n", i, messages[i], ret, GetLastError()); + } + + /* post, topmost */ + ret = PostMessageA(HWND_TOPMOST, messages[i], 0, 0); + ok(ret, "%d: got %d, error %d\n", i, ret, GetLastError()); + + memset(&msg, 0xab, sizeof(msg)); + ret = PeekMessageA(&msg, 0, 0, 0, PM_REMOVE); + if (messages[i] < WM_USER || messages[i] >= 0xc000) + { + ok(ret, "%d: message %04x, got %d, error %d\n", i, messages[i], ret, GetLastError()); + ok(msg.hwnd == hwnd, "%d: got %p\n", i, msg.hwnd); + } + else + { + todo_wine + ok(!ret, "%d: got %d, error %d\n", i, ret, GetLastError()); + } + + /* send, broadcast */ + g_broadcast_lparam = 0xdead; + ret = SendMessageTimeoutA(HWND_BROADCAST, messages[i], 0xbaadbeef, 0, SMTO_NORMAL, 2000, NULL); + if (!ret && GetLastError() == ERROR_TIMEOUT) + win_skip("broadcasting test %d, timeout\n", i); + else + { + if (messages[i] < WM_USER || messages[i] >= 0xc000) + { + ok(g_broadcast_lparam == 0xbaadbeef, "%d: message %04x, got %#lx, error %d\n", i, messages[i], + g_broadcast_lparam, GetLastError()); + } + else + { + todo_wine + ok(g_broadcast_lparam == 0xdead, "%d: message %04x, got %#lx, error %d\n", i, messages[i], + g_broadcast_lparam, GetLastError()); + } + } + + /* send, topmost */ + g_broadcast_lparam = 0xdead; + ret = SendMessageTimeoutA(HWND_TOPMOST, messages[i], 0xbaadbeef, 0, SMTO_NORMAL, 2000, NULL); + if (!ret && GetLastError() == ERROR_TIMEOUT) + win_skip("broadcasting test %d, timeout\n", i); + else + { + if (messages[i] < WM_USER || messages[i] >= 0xc000) + { + ok(g_broadcast_lparam == 0xbaadbeef, "%d: message %04x, got %#lx, error %d\n", i, messages[i], + g_broadcast_lparam, GetLastError()); + } + else + { + todo_wine + ok(g_broadcast_lparam == 0xdead, "%d: message %04x, got %#lx, error %d\n", i, messages[i], + g_broadcast_lparam, GetLastError()); + } + } + } + + DestroyWindow(hwnd); +} + static const struct { DWORD exp, broken; @@ -15642,6 +15763,7 @@ START_TEST(msg) test_SetFocus(); test_SetParent(); test_PostMessage(); + test_broadcast(); test_ShowWindow(); test_PeekMessage(); test_PeekMessage2();

1 0

Michael Müller : kernel32: Implement GetFinalPathNameByHandle.
by Alexandre Julliard April 8, 2016

April 8, 2016

Module: wine Branch: master Commit: 6e12aba9c7f0fef8d2b1b7202ad778c399e6e862 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6e12aba9c7f0fef8d2b1b7202… Author: Michael Müller <michael(a)fds-team.de> Date: Thu Apr 7 10:28:52 2016 -0500 kernel32: Implement GetFinalPathNameByHandle. Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- .../api-ms-win-core-file-l1-1-0.spec | 4 +- .../api-ms-win-core-file-l1-2-0.spec | 4 +- .../api-ms-win-core-file-l1-2-1.spec | 4 +- dlls/kernel32/file.c | 211 +++++++++++++++++++++ dlls/kernel32/kernel32.spec | 4 +- dlls/kernel32/tests/file.c | 40 ++++ dlls/kernelbase/kernelbase.spec | 4 +- 7 files changed, 261 insertions(+), 10 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=6e12aba9c7f0fef8d2b1b…

1 0

Nikolay Sivov : comctl32/listview: Handle DPA_SetPtr() failure (Coverity).
by Alexandre Julliard April 8, 2016

April 8, 2016

Module: wine Branch: master Commit: f5ca9b91046f43410934ddf872bf4504cd9e9fe1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f5ca9b91046f43410934ddf87… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Fri Apr 8 16:56:40 2016 +0300 comctl32/listview: Handle DPA_SetPtr() failure (Coverity). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/comctl32/listview.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/dlls/comctl32/listview.c b/dlls/comctl32/listview.c index b467b36..758b640 100644 --- a/dlls/comctl32/listview.c +++ b/dlls/comctl32/listview.c @@ -3123,7 +3123,11 @@ static RANGES ranges_clone(RANGES ranges) RANGE *newrng = Alloc(sizeof(RANGE)); if (!newrng) goto fail; *newrng = *((RANGE*)DPA_GetPtr(ranges->hdpa, i)); - DPA_SetPtr(clone->hdpa, i, newrng); + if (!DPA_SetPtr(clone->hdpa, i, newrng)) + { + Free(newrng); + goto fail; + } } return clone;

1 0

Michael Stefaniuc : comctl32/tests: Use SetRect() instead of open coding it.
by Alexandre Julliard April 8, 2016

April 8, 2016

Module: wine Branch: master Commit: 87bc2aa60776b89e6a120fb65e6035dc9734822f URL: http://source.winehq.org/git/wine.git/?a=commit;h=87bc2aa60776b89e6a120fb65… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Fri Apr 8 13:38:03 2016 +0200 comctl32/tests: Use SetRect() instead of open coding it. Signed-off-by: Michael Stefaniuc <mstefani(a)redhat.de> Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/comctl32/tests/animate.c | 5 +- dlls/comctl32/tests/listview.c | 132 ++++++++++++----------------------------- dlls/comctl32/tests/monthcal.c | 2 +- dlls/comctl32/tests/progress.c | 7 +-- 4 files changed, 42 insertions(+), 104 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=87bc2aa60776b89e6a120…

1 0

Huw Davies : gdi32: Implement playback of the GradientFill EMF record.
by Alexandre Julliard April 8, 2016

April 8, 2016

Module: wine Branch: master Commit: 66351b3de39cc835be52034c9e528cef93ac8492 URL: http://source.winehq.org/git/wine.git/?a=commit;h=66351b3de39cc835be52034c9… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Apr 8 11:40:57 2016 +0100 gdi32: Implement playback of the GradientFill EMF record. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/enhmetafile.c | 35 +++++++++++++++++++++-------------- 1 file changed, 21 insertions(+), 14 deletions(-) diff --git a/dlls/gdi32/enhmetafile.c b/dlls/gdi32/enhmetafile.c index 7fb0b34..995997c 100644 --- a/dlls/gdi32/enhmetafile.c +++ b/dlls/gdi32/enhmetafile.c @@ -2175,23 +2175,30 @@ BOOL WINAPI PlayEnhMetaFileRecord( break; } + case EMR_GRADIENTFILL: + { + EMRGRADIENTFILL *grad = (EMRGRADIENTFILL *)mr; + GdiGradientFill( hdc, grad->Ver, grad->nVer, grad->Ver + grad->nVer, + grad->nTri, grad->ulMode ); + break; + } + case EMR_POLYDRAW16: case EMR_GLSRECORD: case EMR_GLSBOUNDEDRECORD: - case EMR_DRAWESCAPE : - case EMR_EXTESCAPE: - case EMR_STARTDOC: - case EMR_SMALLTEXTOUT: - case EMR_FORCEUFIMAPPING: - case EMR_NAMEDESCAPE: - case EMR_COLORCORRECTPALETTE: - case EMR_SETICMPROFILEA: - case EMR_SETICMPROFILEW: - case EMR_TRANSPARENTBLT: - case EMR_GRADIENTFILL: - case EMR_SETLINKEDUFI: - case EMR_COLORMATCHTOTARGETW: - case EMR_CREATECOLORSPACEW: + case EMR_DRAWESCAPE: + case EMR_EXTESCAPE: + case EMR_STARTDOC: + case EMR_SMALLTEXTOUT: + case EMR_FORCEUFIMAPPING: + case EMR_NAMEDESCAPE: + case EMR_COLORCORRECTPALETTE: + case EMR_SETICMPROFILEA: + case EMR_SETICMPROFILEW: + case EMR_TRANSPARENTBLT: + case EMR_SETLINKEDUFI: + case EMR_COLORMATCHTOTARGETW: + case EMR_CREATECOLORSPACEW: default: /* From docs: If PlayEnhMetaFileRecord doesn't recognize a

1 0

Henri Verbeet : wined3d: Use the texture dimension helpers in surface_load_ds_location().
by Alexandre Julliard April 8, 2016

April 8, 2016

Module: wine Branch: master Commit: 7c9ff85452003ad6ea1b45c9b0d405816b3a3b29 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7c9ff85452003ad6ea1b45c9b… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Apr 7 19:38:00 2016 +0200 wined3d: Use the texture dimension helpers in surface_load_ds_location(). Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/surface.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 5fd238e..8ac3b2f 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -2820,8 +2820,8 @@ static void surface_load_ds_location(struct wined3d_surface *surface, struct win } else { - w = surface->resource.width; - h = surface->resource.height; + w = wined3d_texture_get_level_width(surface->container, surface->texture_level); + h = wined3d_texture_get_level_height(surface->container, surface->texture_level); } if (surface->current_renderbuffer)

1 0

Huw Davies : gdi32: Implement recording of the GradientFill EMF record.
by Alexandre Julliard April 8, 2016

April 8, 2016

Module: wine Branch: master Commit: ac0959cdf63ab67269bd388df75555f1d83cfa86 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ac0959cdf63ab67269bd388df… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Apr 8 11:40:56 2016 +0100 gdi32: Implement recording of the GradientFill EMF record. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/enhmfdrv/enhmetafiledrv.h | 2 + dlls/gdi32/enhmfdrv/graphics.c | 54 +++++++++++++++++++++++ dlls/gdi32/enhmfdrv/init.c | 2 +- dlls/gdi32/tests/metafile.c | 83 ++++++++++++++++++++++++++++++++++++ 4 files changed, 140 insertions(+), 1 deletion(-) diff --git a/dlls/gdi32/enhmfdrv/enhmetafiledrv.h b/dlls/gdi32/enhmfdrv/enhmetafiledrv.h index cbd7368..721c82f 100644 --- a/dlls/gdi32/enhmfdrv/enhmetafiledrv.h +++ b/dlls/gdi32/enhmfdrv/enhmetafiledrv.h @@ -75,6 +75,8 @@ extern BOOL EMFDRV_FlattenPath( PHYSDEV dev ) DECLSPEC_HIDDEN; extern BOOL EMFDRV_FrameRgn( PHYSDEV dev, HRGN hrgn, HBRUSH hbrush, INT width, INT height ) DECLSPEC_HIDDEN; extern BOOL EMFDRV_GdiComment( PHYSDEV dev, UINT bytes, const BYTE *buffer ) DECLSPEC_HIDDEN; extern INT EMFDRV_GetDeviceCaps( PHYSDEV dev, INT cap ) DECLSPEC_HIDDEN; +extern BOOL EMFDRV_GradientFill( PHYSDEV dev, TRIVERTEX *vert_array, ULONG nvert, + void *grad_array, ULONG ngrad, ULONG mode ) DECLSPEC_HIDDEN; extern INT EMFDRV_IntersectClipRect( PHYSDEV dev, INT left, INT top, INT right, INT bottom ) DECLSPEC_HIDDEN; extern BOOL EMFDRV_InvertRgn( PHYSDEV dev, HRGN hrgn ) DECLSPEC_HIDDEN; extern BOOL EMFDRV_LineTo( PHYSDEV dev, INT x, INT y ) DECLSPEC_HIDDEN; diff --git a/dlls/gdi32/enhmfdrv/graphics.c b/dlls/gdi32/enhmfdrv/graphics.c index d3b3325..ae05378 100644 --- a/dlls/gdi32/enhmfdrv/graphics.c +++ b/dlls/gdi32/enhmfdrv/graphics.c @@ -855,3 +855,57 @@ no_bounds: HeapFree( GetProcessHeap(), 0, pemr ); return ret; } + +/********************************************************************** + * EMFDRV_GradientFill + */ +BOOL EMFDRV_GradientFill( PHYSDEV dev, TRIVERTEX *vert_array, ULONG nvert, + void *grad_array, ULONG ngrad, ULONG mode ) +{ + EMRGRADIENTFILL *emr; + ULONG i, pt, size, num_pts = ngrad * (mode == GRADIENT_FILL_TRIANGLE ? 3 : 2); + const ULONG *pts = (const ULONG *)grad_array; + BOOL ret; + + size = FIELD_OFFSET(EMRGRADIENTFILL, Ver[nvert]) + num_pts * sizeof(pts[0]); + + emr = HeapAlloc( GetProcessHeap(), 0, size ); + if (!emr) return FALSE; + + for (i = 0; i < num_pts; i++) + { + pt = pts[i]; + + if (i == 0) + { + emr->rclBounds.left = emr->rclBounds.right = vert_array[pt].x; + emr->rclBounds.top = emr->rclBounds.bottom = vert_array[pt].y; + } + else + { + if (vert_array[pt].x < emr->rclBounds.left) + emr->rclBounds.left = vert_array[pt].x; + else if (vert_array[pt].x > emr->rclBounds.right) + emr->rclBounds.right = vert_array[pt].x; + if (vert_array[pt].y < emr->rclBounds.top) + emr->rclBounds.top = vert_array[pt].y; + else if (vert_array[pt].y > emr->rclBounds.bottom) + emr->rclBounds.bottom = vert_array[pt].y; + } + } + emr->rclBounds.right--; + emr->rclBounds.bottom--; + + emr->emr.iType = EMR_GRADIENTFILL; + emr->emr.nSize = size; + emr->nVer = nvert; + emr->nTri = ngrad; + emr->ulMode = mode; + memcpy( emr->Ver, vert_array, nvert * sizeof(vert_array[0]) ); + memcpy( emr->Ver + nvert, pts, num_pts * sizeof(pts[0]) ); + + EMFDRV_UpdateBBox( dev, &emr->rclBounds ); + ret = EMFDRV_WriteRecord( dev, &emr->emr ); + HeapFree( GetProcessHeap(), 0, emr ); + return ret; +} diff --git a/dlls/gdi32/enhmfdrv/init.c b/dlls/gdi32/enhmfdrv/init.c index 34b3de6..e0cbcae 100644 --- a/dlls/gdi32/enhmfdrv/init.c +++ b/dlls/gdi32/enhmfdrv/init.c @@ -92,7 +92,7 @@ static const struct gdi_dc_funcs EMFDRV_Funcs = NULL, /* pGetTextExtentExPointI */ NULL, /* pGetTextFace */ NULL, /* pGetTextMetrics */ - NULL, /* pGradientFill */ + EMFDRV_GradientFill, /* pGradientFill */ EMFDRV_IntersectClipRect, /* pIntersectClipRect */ EMFDRV_InvertRgn, /* pInvertRgn */ EMFDRV_LineTo, /* pLineTo */ diff --git a/dlls/gdi32/tests/metafile.c b/dlls/gdi32/tests/metafile.c index 5b3bd9f..68ac425 100644 --- a/dlls/gdi32/tests/metafile.c +++ b/dlls/gdi32/tests/metafile.c @@ -1424,6 +1424,44 @@ static const unsigned char EMF_POLYPOLYLINE_BITS[] = 0x14, 0x00, 0x00, 0x00 }; +static const unsigned char EMF_GRADIENTFILL_BITS[] = +{ + 0x01, 0x00, 0x00, 0x00, 0x6c, 0x00, 0x00, 0x00, + 0x01, 0x00, 0x00, 0x00, 0x0a, 0x00, 0x00, 0x00, + 0x2b, 0x01, 0x00, 0x00, 0x35, 0x01, 0x00, 0x00, + 0x23, 0x00, 0x00, 0x00, 0x61, 0x01, 0x00, 0x00, + 0x31, 0x29, 0x00, 0x00, 0xa3, 0x2a, 0x00, 0x00, + 0x20, 0x45, 0x4d, 0x46, 0x00, 0x00, 0x01, 0x00, + 0x0c, 0x01, 0x00, 0x00, 0x03, 0x00, 0x00, 0x00, + 0x01, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, + 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, + 0x40, 0x05, 0x00, 0x00, 0x46, 0x03, 0x00, 0x00, + 0xda, 0x01, 0x00, 0x00, 0x28, 0x01, 0x00, 0x00, + 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, + 0x00, 0x00, 0x00, 0x00, 0x15, 0x3c, 0x07, 0x00, + 0xcb, 0x82, 0x04, 0x00, 0x76, 0x00, 0x00, 0x00, + 0x8c, 0x00, 0x00, 0x00, 0x01, 0x00, 0x00, 0x00, + 0x0a, 0x00, 0x00, 0x00, 0x2b, 0x01, 0x00, 0x00, + 0x35, 0x01, 0x00, 0x00, 0x05, 0x00, 0x00, 0x00, + 0x02, 0x00, 0x00, 0x00, 0x02, 0x00, 0x00, 0x00, + 0x01, 0x00, 0x00, 0x00, 0x0a, 0x00, 0x00, 0x00, + 0x00, 0xff, 0x00, 0x80, 0x00, 0x00, 0x01, 0x80, + 0xc8, 0x00, 0x00, 0x00, 0xd2, 0x00, 0x00, 0x00, + 0x00, 0x00, 0x00, 0x00, 0x00, 0xff, 0x00, 0xff, + 0xb4, 0x00, 0x00, 0x00, 0xbe, 0x00, 0x00, 0x00, + 0x34, 0x12, 0x78, 0x56, 0xbc, 0x9a, 0xf0, 0xde, + 0x2c, 0x01, 0x00, 0x00, 0x36, 0x01, 0x00, 0x00, + 0x00, 0xff, 0x00, 0xff, 0x00, 0xff, 0x00, 0x00, + 0x90, 0x01, 0x00, 0x00, 0x9a, 0x01, 0x00, 0x00, + 0x00, 0xff, 0x00, 0xff, 0x00, 0xff, 0x00, 0x00, + 0x00, 0x00, 0x00, 0x00, 0x01, 0x00, 0x00, 0x00, + 0x02, 0x00, 0x00, 0x00, 0x03, 0x00, 0x00, 0x00, + 0x01, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, + 0x0e, 0x00, 0x00, 0x00, 0x14, 0x00, 0x00, 0x00, + 0x00, 0x00, 0x00, 0x00, 0x10, 0x00, 0x00, 0x00, + 0x14, 0x00, 0x00, 0x00 +}; + /* For debugging or dumping the raw metafiles produced by * new test functions. */ @@ -3599,6 +3637,50 @@ static void test_emf_PolyPolyline(void) DeleteEnhMetaFile(hemf); } +static void test_emf_GradientFill(void) +{ + HDC mf; + HENHMETAFILE hemf; + TRIVERTEX v[] = + { + { 1, 10, 0xff00, 0x8000, 0x0000, 0x8001 }, + { 200, 210, 0x0000, 0x0000, 0xff00, 0xff00 }, + { 180, 190, 0x1234, 0x5678, 0x9abc, 0xdef0 }, + { 300, 310, 0xff00, 0xff00, 0xff00, 0x0000 }, + { 400, 410, 0xff00, 0xff00, 0xff00, 0x0000 } + }; + GRADIENT_TRIANGLE tri[] = { { 0, 1, 2 }, { 3, 1, 0 } }; + BOOL ret; + + mf = CreateEnhMetaFileA( GetDC( 0 ), NULL, NULL, NULL ); + ok( mf != 0, "CreateEnhMetaFileA error %d\n", GetLastError() ); + + /* Don't test the GRADIENT_FILL_RECT_ modes since a Windows bug + * means it allocates three mesh indicies rather than two per + * rectangle. This results in uninitialised values being written + * to the EMF which is rather difficult to test against. + * + * Note also that the final vertex here is not required, yet it is + * written to the EMF, but is not considered in the bounds + * calculation. + */ + ret = GdiGradientFill( mf, v, sizeof(v) / sizeof(v[0]), tri, sizeof(tri) / sizeof(tri[0]), + GRADIENT_FILL_TRIANGLE ); + ok( ret, "GradientFill\n" ); + + hemf = CloseEnhMetaFile( mf ); + ok( hemf != 0, "CloseEnhMetaFile error %d\n", GetLastError() ); + + if (compare_emf_bits( hemf, EMF_GRADIENTFILL_BITS, sizeof(EMF_GRADIENTFILL_BITS), + "emf_GradientFill", FALSE ) != 0) + { + dump_emf_bits( hemf, "emf_GradientFill" ); + dump_emf_records( hemf, "emf_GradientFill" ); + } + + DeleteEnhMetaFile( hemf ); +} + START_TEST(metafile) { init_function_pointers(); @@ -3614,6 +3696,7 @@ START_TEST(metafile) test_emf_polybezier(); test_emf_GetPath(); test_emf_PolyPolyline(); + test_emf_GradientFill(); /* For win-format metafiles (mfdrv) */ test_mf_SaveDC();

1 0