December 2011 - Wine-Commits

Dmitry Timoshkov : user32: Add some SetFocus message tests.
by Alexandre Julliard Dec. 30, 2011

Dec. 30, 2011

Module: wine Branch: master Commit: 229fb8a4478fa4aa9ed55b697938fd11c066b2ee URL: http://source.winehq.org/git/wine.git/?a=commit;h=229fb8a4478fa4aa9ed55b697… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Fri Dec 30 15:36:19 2011 +0800 user32: Add some SetFocus message tests. --- dlls/user32/tests/msg.c | 100 +++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 100 insertions(+), 0 deletions(-) diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index 1a02e02..407fc7b 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -13490,6 +13490,105 @@ end: flush_sequence(); } + +static const struct message WmSetFocus_1[] = { + { HCBT_SETFOCUS, hook }, /* child */ + { HCBT_ACTIVATE, hook }, /* parent */ + { WM_QUERYNEWPALETTE, sent|wparam|lparam|parent|optional, 0, 0 }, + { WM_WINDOWPOSCHANGING, sent|parent, 0, SWP_NOSIZE|SWP_NOMOVE }, + { WM_ACTIVATEAPP, sent|wparam|parent, 1 }, + { WM_NCACTIVATE, sent|wparam|parent, 1 }, + { WM_GETTEXT, sent|defwinproc|parent|optional }, + { WM_GETTEXT, sent|defwinproc|parent|optional }, + { WM_ACTIVATE, sent|wparam|parent, 1 }, + { HCBT_SETFOCUS, hook }, /* parent */ + { WM_SETFOCUS, sent|defwinproc|parent }, + { WM_KILLFOCUS, sent|parent }, + { WM_SETFOCUS, sent }, + { 0 } +}; +static const struct message WmSetFocus_2[] = { + { HCBT_SETFOCUS, hook }, /* parent */ + { WM_KILLFOCUS, sent }, + { WM_SETFOCUS, sent|parent }, + { 0 } +}; +static const struct message WmSetFocus_3[] = { + { HCBT_SETFOCUS, hook }, /* child */ + { 0 } +}; +static const struct message WmSetFocus_4[] = { + { 0 } +}; + +static void test_SetFocus(void) +{ + HWND parent, old_parent, child, old_focus; + MSG msg; + + parent = CreateWindowEx(0, "TestParentClass", NULL, WS_OVERLAPPEDWINDOW, + 0, 0, 0, 0, 0, 0, 0, NULL); + ok(parent != 0, "failed to create parent window\n"); + child = CreateWindowEx(0, "TestWindowClass", NULL, WS_CHILD, + 0, 0, 0, 0, parent, 0, 0, NULL); + ok(child != 0, "failed to create child window\n"); + + trace("parent %p, child %p\n", parent, child); + + SetFocus(0); + SetActiveWindow(0); + + flush_events(); + flush_sequence(); + + ok(GetActiveWindow() == 0, "expected active 0, got %p\n", GetActiveWindow()); + ok(GetFocus() == 0, "expected focus 0, got %p\n", GetFocus()); + + log_all_parent_messages++; + + old_focus = SetFocus(child); + while (PeekMessage(&msg, NULL, 0, 0, PM_REMOVE)) DispatchMessage(&msg); + ok_sequence(WmSetFocus_1, "SetFocus on a child window", TRUE); + ok(old_focus == parent, "expected old focus %p, got %p\n", parent, old_focus); + ok(GetActiveWindow() == parent, "expected active %p, got %p\n", parent, GetActiveWindow()); + ok(GetFocus() == child, "expected focus %p, got %p\n", child, GetFocus()); + + old_focus = SetFocus(parent); + while (PeekMessage(&msg, NULL, 0, 0, PM_REMOVE)) DispatchMessage(&msg); + ok_sequence(WmSetFocus_2, "SetFocus on a parent window", FALSE); + ok(old_focus == child, "expected old focus %p, got %p\n", child, old_focus); + ok(GetActiveWindow() == parent, "expected active %p, got %p\n", parent, GetActiveWindow()); + ok(GetFocus() == parent, "expected focus %p, got %p\n", parent, GetFocus()); + + old_parent = SetParent(child, GetDesktopWindow()); + ok(old_parent == parent, "expected old parent %p, got %p\n", parent, old_parent); + + ok(GetActiveWindow() == parent, "expected active %p, got %p\n", parent, GetActiveWindow()); + ok(GetFocus() == parent, "expected focus %p, got %p\n", parent, GetFocus()); + + flush_events(); + flush_sequence(); + + old_focus = SetFocus(child); + while (PeekMessage(&msg, NULL, 0, 0, PM_REMOVE)) DispatchMessage(&msg); + ok_sequence(WmSetFocus_3, "SetFocus on a child window", TRUE); + ok(old_focus == 0, "expected old focus 0, got %p\n", old_focus); + ok(GetActiveWindow() == parent, "expected active %p, got %p\n", parent, GetActiveWindow()); + ok(GetFocus() == parent, "expected focus %p, got %p\n", parent, GetFocus()); + + old_focus = SetActiveWindow(child); + while (PeekMessage(&msg, NULL, 0, 0, PM_REMOVE)) DispatchMessage(&msg); + ok_sequence(WmEmptySeq, "SetActiveWindow on a child window", FALSE); + ok(old_focus == parent, "expected old active %p, got %p\n", parent, old_focus); + ok(GetActiveWindow() == parent, "expected active %p, got %p\n", parent, GetActiveWindow()); + ok(GetFocus() == parent, "expected focus %p, got %p\n", parent, GetFocus()); + + log_all_parent_messages--; + + DestroyWindow(child); + DestroyWindow(parent); +} + START_TEST(msg) { char **test_argv; @@ -13553,6 +13652,7 @@ START_TEST(msg) hEvent_hook = 0; #endif + test_SetFocus(); test_SetParent(); test_PostMessage(); test_ShowWindow();

1 0

Francois Gouget : include: Add some D3D import and include directives.
by Alexandre Julliard Dec. 30, 2011

Dec. 30, 2011

Module: wine Branch: master Commit: 5cb365bac0807b2f656a199a748d945b9aa27849 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5cb365bac0807b2f656a199a7… Author: Francois Gouget <fgouget(a)free.fr> Date: Fri Dec 30 09:31:53 2011 +0100 include: Add some D3D import and include directives. --- include/d3d10.idl | 4 +++- include/d3d10_1.idl | 2 ++ include/d3d11.idl | 10 ++++++++++ 3 files changed, 15 insertions(+), 1 deletions(-) diff --git a/include/d3d10.idl b/include/d3d10.idl index 51b92bd..ac98942 100644 --- a/include/d3d10.idl +++ b/include/d3d10.idl @@ -1592,7 +1592,9 @@ interface ID3D10Multithread : IUnknown BOOL GetMultithreadProtected(); } +cpp_quote("#ifndef D3D10_IGNORE_SDK_LAYERS") +cpp_quote("/* FIXME: # include <d3d10sdklayers.h> */") +cpp_quote("#endif") cpp_quote("#include \"d3d10misc.h\"") cpp_quote("#include \"d3d10shader.h\"") cpp_quote("#include \"d3d10effect.h\"") -/* TODO: Include "d310sdklayers.h" as soon as it exists */ diff --git a/include/d3d10_1.idl b/include/d3d10_1.idl index 5724b63..11c52b4 100644 --- a/include/d3d10_1.idl +++ b/include/d3d10_1.idl @@ -20,6 +20,8 @@ import "oaidl.idl"; import "ocidl.idl"; import "d3d10.idl"; +cpp_quote("#include <d3d10_1shader.h>") + typedef enum D3D10_FEATURE_LEVEL1 { diff --git a/include/d3d11.idl b/include/d3d11.idl index 5959010..80b929c 100644 --- a/include/d3d11.idl +++ b/include/d3d11.idl @@ -21,6 +21,16 @@ import "ocidl.idl"; import "dxgi.idl"; import "d3dcommon.idl"; +cpp_quote("#include <d3d10_1.h>") +cpp_quote("#ifndef D3D11_IGNORE_SDK_LAYERS") +cpp_quote("/* FIXME: # include <d3d11sdklayers.h> */") +cpp_quote("#endif") +cpp_quote("#include <d3d10misc.h>") +cpp_quote("#include <d3d10shader.h>") +cpp_quote("#include <d3d10effect.h>") +cpp_quote("#include <d3d10_1shader.h>") + + typedef D3D_SRV_DIMENSION D3D11_SRV_DIMENSION; typedef D3D_PRIMITIVE D3D11_PRIMITIVE;

1 0

Alexandre Julliard : gdi32: Don' t bother computing interior regions when the brush is null.
by Alexandre Julliard Dec. 30, 2011

Dec. 30, 2011

Module: wine Branch: master Commit: 2c9c9f78fb02abb1dd28ddf7a0d100d4a34e00e7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2c9c9f78fb02abb1dd28ddf7a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Dec 30 12:53:19 2011 +0100 gdi32: Don't bother computing interior regions when the brush is null. --- dlls/gdi32/dibdrv/graphics.c | 38 ++++++++++++++++++++++++++------------ 1 files changed, 26 insertions(+), 12 deletions(-) diff --git a/dlls/gdi32/dibdrv/graphics.c b/dlls/gdi32/dibdrv/graphics.c index 26d3d58..d14148e 100644 --- a/dlls/gdi32/dibdrv/graphics.c +++ b/dlls/gdi32/dibdrv/graphics.c @@ -527,9 +527,9 @@ BOOL dibdrv_PolyPolygon( PHYSDEV dev, const POINT *pt, const INT *counts, DWORD dibdrv_physdev *pdev = get_dibdrv_pdev(dev); PHYSDEV next = GET_NEXT_PHYSDEV( dev, pPolyPolygon ); DWORD total, i, pos; - BOOL ret = FALSE; + BOOL ret = TRUE; POINT *points; - HRGN outline = 0, interior; + HRGN outline = 0, interior = 0; if (defer_pen( pdev )) return next->funcs->pPolyPolygon( next, pt, counts, polygons ); @@ -544,15 +544,22 @@ BOOL dibdrv_PolyPolygon( PHYSDEV dev, const POINT *pt, const INT *counts, DWORD memcpy( points, pt, total * sizeof(*pt) ); LPtoDP( dev->hdc, points, total ); - if (!(interior = CreatePolyPolygonRgn( points, counts, polygons, GetPolyFillMode( dev->hdc )))) + if (pdev->brush.style != BS_NULL && + !(interior = CreatePolyPolygonRgn( points, counts, polygons, GetPolyFillMode( dev->hdc )))) { HeapFree( GetProcessHeap(), 0, points ); return FALSE; } + if (pdev->pen_uses_region) outline = CreateRectRgn( 0, 0, 0, 0 ); + /* if not using a region, paint the interior first so the outline can overlap it */ - if (!pdev->pen_uses_region || !(outline = CreateRectRgn( 0, 0, 0, 0 ))) + if (interior && !outline) + { ret = brush_region( pdev, interior ); + DeleteObject( interior ); + interior = 0; + } for (i = pos = 0; i < polygons; i++) { @@ -561,14 +568,17 @@ BOOL dibdrv_PolyPolygon( PHYSDEV dev, const POINT *pt, const INT *counts, DWORD pos += counts[i]; } - if (outline) + if (interior) { CombineRgn( interior, interior, outline, RGN_DIFF ); - ret = pen_region( pdev, outline ) && brush_region( pdev, interior ); + ret = brush_region( pdev, interior ); + DeleteObject( interior ); + } + if (outline) + { + if (ret) ret = pen_region( pdev, outline ); DeleteObject( outline ); } - - DeleteObject( interior ); HeapFree( GetProcessHeap(), 0, points ); return ret; } @@ -687,12 +697,16 @@ BOOL dibdrv_Rectangle( PHYSDEV dev, INT left, INT top, INT right, INT bottom ) if (outline) { - HRGN interior = CreateRectRgnIndirect( &rect ); + if (pdev->brush.style != BS_NULL) + { + HRGN interior = CreateRectRgnIndirect( &rect ); - CombineRgn( interior, interior, outline, RGN_DIFF ); - ret = pen_region( pdev, outline ) && brush_region( pdev, interior ); + CombineRgn( interior, interior, outline, RGN_DIFF ); + brush_region( pdev, interior ); + DeleteObject( interior ); + } + ret = pen_region( pdev, outline ); DeleteObject( outline ); - DeleteObject( interior ); } else {

1 0

Alexandre Julliard : user32/tests: Remove a crashing test.
by Alexandre Julliard Dec. 30, 2011

Dec. 30, 2011

Module: wine Branch: master Commit: f6ccb2c498f79099d1291d3c706944f846c7e530 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f6ccb2c498f79099d1291d3c7… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Dec 30 10:17:00 2011 +0100 user32/tests: Remove a crashing test. --- dlls/user32/tests/msg.c | 27 --------------------------- 1 files changed, 0 insertions(+), 27 deletions(-) diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index bde0023..1a02e02 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -8134,12 +8134,6 @@ static VOID CALLBACK tfunc(HWND hwnd, UINT uMsg, UINT_PTR id, DWORD dwTime) { } -static VOID CALLBACK tfunc_crash(HWND hwnd, UINT uMsg, UINT_PTR id, DWORD dwTime) -{ - /* Crash on purpose */ - *(volatile int *)0 = 2; -} - #define TIMER_ID 0x19 static DWORD WINAPI timer_thread_proc(LPVOID x) @@ -8161,7 +8155,6 @@ static void test_timers(void) { struct timer_info info; DWORD id; - MSG msg; info.hWnd = CreateWindow ("TestWindowClass", NULL, WS_OVERLAPPEDWINDOW , @@ -8184,26 +8177,6 @@ static void test_timers(void) ok( KillTimer(info.hWnd, TIMER_ID), "KillTimer failed\n"); ok(DestroyWindow(info.hWnd), "failed to destroy window\n"); - - /* Test timer callback with crash */ - SetLastError(0xdeadbeef); - info.hWnd = CreateWindowW(testWindowClassW, NULL, - WS_OVERLAPPEDWINDOW , - CW_USEDEFAULT, CW_USEDEFAULT, 300, 300, 0, - NULL, NULL, 0); - if ((!info.hWnd && GetLastError() == ERROR_CALL_NOT_IMPLEMENTED) || /* Win9x/Me */ - (!pGetMenuInfo)) /* Win95/NT4 */ - { - win_skip("Test would crash on Win9x/WinMe/NT4\n"); - DestroyWindow(info.hWnd); - return; - } - info.id = SetTimer(info.hWnd, TIMER_ID, 0, tfunc_crash); - ok(info.id, "SetTimer failed\n"); - Sleep(150); - while (PeekMessage(&msg, NULL, 0, 0, PM_REMOVE)) DispatchMessage(&msg); - - ok(DestroyWindow(info.hWnd), "failed to destroy window\n"); } static int count = 0;

1 0

Alexandre Julliard : gdi32: Implement brushed pens.
by Alexandre Julliard Dec. 30, 2011

Dec. 30, 2011

Module: wine Branch: master Commit: 488bf107a9623c94442a96b515a2b1e853c3c7a1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=488bf107a9623c94442a96b51… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Dec 30 10:38:18 2011 +0100 gdi32: Implement brushed pens. --- dlls/gdi32/dibdrv/dc.c | 8 +- dlls/gdi32/dibdrv/dibdrv.h | 2 +- dlls/gdi32/dibdrv/objects.c | 276 ++++++++++++++++++++----------------------- 3 files changed, 137 insertions(+), 149 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=488bf107a9623c94442a9…

1 0

Alexandre Julliard : gdi32: Define a structure to make the brush state independent from the physdev.
by Alexandre Julliard Dec. 30, 2011

Dec. 30, 2011

Module: wine Branch: master Commit: 4e03247427dc23af776d9a14652bf338cc27e5f6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4e03247427dc23af776d9a146… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Dec 30 10:17:37 2011 +0100 gdi32: Define a structure to make the brush state independent from the physdev. --- dlls/gdi32/dibdrv/bitblt.c | 4 +- dlls/gdi32/dibdrv/dc.c | 6 +- dlls/gdi32/dibdrv/dibdrv.h | 31 +++++--- dlls/gdi32/dibdrv/graphics.c | 6 +- dlls/gdi32/dibdrv/objects.c | 175 +++++++++++++++++++++--------------------- 5 files changed, 116 insertions(+), 106 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=4e03247427dc23af776d9…

1 0

Alexandre Julliard : gdi32: Pass a brush_pattern structure to the SelectPen entry point for brushed pens .
by Alexandre Julliard Dec. 30, 2011

Dec. 30, 2011

Module: wine Branch: master Commit: cc9836bda02798ba48383acae7acb1e9899cd4b7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=cc9836bda02798ba48383acae… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 29 20:16:46 2011 +0100 gdi32: Pass a brush_pattern structure to the SelectPen entry point for brushed pens. --- dlls/gdi32/brush.c | 48 +++++++++++++++---------------- dlls/gdi32/dibdrv/dibdrv.h | 2 +- dlls/gdi32/dibdrv/objects.c | 4 +- dlls/gdi32/driver.c | 2 +- dlls/gdi32/enhmfdrv/enhmetafiledrv.h | 2 +- dlls/gdi32/enhmfdrv/objects.c | 2 +- dlls/gdi32/gdi_private.h | 1 + dlls/gdi32/mfdrv/metafiledrv.h | 2 +- dlls/gdi32/mfdrv/objects.c | 2 +- dlls/gdi32/pen.c | 51 ++++++++++++++++++++++++---------- dlls/wineps.drv/pen.c | 2 +- dlls/wineps.drv/psdrv.h | 2 +- dlls/winex11.drv/pen.c | 2 +- dlls/winex11.drv/x11drv.h | 2 +- include/wine/gdi_driver.h | 4 +- 15 files changed, 74 insertions(+), 54 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=cc9836bda02798ba48383…

1 0

Alexandre Julliard : gdi32: Pass a brush_pattern structure to the SelectBrush entry point.
by Alexandre Julliard Dec. 30, 2011

Dec. 30, 2011

Module: wine Branch: master Commit: f85bb3528047fc3a5fd2291dea9e21ed72b8a843 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f85bb3528047fc3a5fd2291de… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 29 19:49:41 2011 +0100 gdi32: Pass a brush_pattern structure to the SelectBrush entry point. --- dlls/gdi32/brush.c | 19 +++++++------------ dlls/gdi32/dibdrv/dibdrv.h | 8 ++------ dlls/gdi32/dibdrv/objects.c | 28 ++++++++++++---------------- dlls/gdi32/driver.c | 3 +-- dlls/gdi32/enhmfdrv/enhmetafiledrv.h | 3 +-- dlls/gdi32/enhmfdrv/objects.c | 3 +-- dlls/gdi32/gdi_private.h | 8 -------- dlls/gdi32/mfdrv/metafiledrv.h | 3 +-- dlls/gdi32/mfdrv/objects.c | 3 +-- dlls/wineps.drv/brush.c | 11 ++++------- dlls/wineps.drv/psdrv.h | 11 ++++------- dlls/winex11.drv/brush.c | 14 ++++++++------ dlls/winex11.drv/x11drv.h | 5 ++--- dlls/winex11.drv/xrender.c | 11 ++++++----- include/wine/gdi_driver.h | 12 ++++++++++-- 15 files changed, 60 insertions(+), 82 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=f85bb3528047fc3a5fd22…

1 0

Alexandre Julliard : gdi32: Store the pattern brush information for geometric pens.
by Alexandre Julliard Dec. 30, 2011

Dec. 30, 2011

Module: wine Branch: master Commit: a1fdd5856f62bb5c9b3efe9d5ca287c0c0858bde URL: http://source.winehq.org/git/wine.git/?a=commit;h=a1fdd5856f62bb5c9b3efe9d5… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 29 16:18:07 2011 +0100 gdi32: Store the pattern brush information for geometric pens. --- dlls/gdi32/pen.c | 72 +++++++++++----------------- dlls/gdi32/tests/pen.c | 121 ++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 150 insertions(+), 43 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=a1fdd5856f62bb5c9b3ef…

1 0

Alexandre Julliard : gdi32: Add a structure to store all the extra information needed for a pattern brush .
by Alexandre Julliard Dec. 30, 2011

Dec. 30, 2011

Module: wine Branch: master Commit: 8bf48557ef7ed0191e1bbd4a495a6fefc2457d3c URL: http://source.winehq.org/git/wine.git/?a=commit;h=8bf48557ef7ed0191e1bbd4a4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 29 16:17:44 2011 +0100 gdi32: Add a structure to store all the extra information needed for a pattern brush. --- dlls/gdi32/brush.c | 142 ++++++++++++++++++++++++---------------------- dlls/gdi32/gdi_private.h | 10 +++ 2 files changed, 85 insertions(+), 67 deletions(-) diff --git a/dlls/gdi32/brush.c b/dlls/gdi32/brush.c index 384ef12..5dc7dfe 100644 --- a/dlls/gdi32/brush.c +++ b/dlls/gdi32/brush.c @@ -36,10 +36,7 @@ typedef struct { GDIOBJHDR header; LOGBRUSH logbrush; - HBITMAP bitmap; /* bitmap handle for DDB pattern brushes */ - BITMAPINFO *info; /* DIB info for pattern brushes */ - struct gdi_image_bits bits; /* DIB bits for pattern brushes */ - UINT usage; /* color usage for DIB info */ + struct brush_pattern pattern; } BRUSHOBJ; #define NB_HATCH_STYLES 6 @@ -58,8 +55,8 @@ static const struct gdi_obj_funcs brush_funcs = }; -/* fetch the contents of the brush bitmap and cache them in the brush object */ -static BOOL store_bitmap_bits( BRUSHOBJ *brush, BITMAPOBJ *bmp ) +/* fetch the contents of the brush bitmap and cache them in the brush pattern */ +static BOOL store_bitmap_bits( struct brush_pattern *brush, BITMAPOBJ *bmp ) { const struct gdi_dc_funcs *funcs = get_bitmap_funcs( bmp ); struct gdi_image_bits bits; @@ -88,7 +85,7 @@ static BOOL store_bitmap_bits( BRUSHOBJ *brush, BITMAPOBJ *bmp ) return TRUE; } -static BOOL copy_bitmap( BRUSHOBJ *brush, HBITMAP bitmap ) +static BOOL copy_bitmap( struct brush_pattern *brush, HBITMAP bitmap ) { BITMAPINFO *info; BITMAPOBJ *bmp = GDI_GetObjPtr( bitmap, OBJ_BITMAP ); @@ -140,6 +137,58 @@ done: return brush->info != NULL; } +BOOL store_brush_pattern( LOGBRUSH *brush, struct brush_pattern *pattern ) +{ + HGLOBAL hmem = 0; + + pattern->bitmap = 0; + pattern->info = NULL; + pattern->bits.free = NULL; + + switch (brush->lbStyle) + { + case BS_SOLID: + case BS_HOLLOW: + case BS_HATCHED: + return TRUE; + + case BS_PATTERN8X8: + brush->lbStyle = BS_PATTERN; + /* fall through */ + case BS_PATTERN: + brush->lbColor = 0; + return copy_bitmap( pattern, (HBITMAP)brush->lbHatch ); + + case BS_DIBPATTERN: + hmem = (HGLOBAL)brush->lbHatch; + if (!(brush->lbHatch = (ULONG_PTR)GlobalLock( hmem ))) return FALSE; + /* fall through */ + case BS_DIBPATTERNPT: + pattern->usage = brush->lbColor; + pattern->info = copy_packed_dib( (BITMAPINFO *)brush->lbHatch, pattern->usage ); + if (hmem) GlobalUnlock( hmem ); + if (!pattern->info) return FALSE; + pattern->bits.ptr = (char *)pattern->info + get_dib_info_size( pattern->info, pattern->usage ); + brush->lbStyle = BS_DIBPATTERN; + brush->lbColor = 0; + return TRUE; + + case BS_DIBPATTERN8X8: + case BS_MONOPATTERN: + case BS_INDEXED: + default: + WARN( "invalid brush style %u\n", brush->lbStyle ); + return FALSE; + } +} + +void free_brush_pattern( struct brush_pattern *pattern ) +{ + if (pattern->bits.free) pattern->bits.free( &pattern->bits ); + if (pattern->bitmap) DeleteObject( pattern->bitmap ); + HeapFree( GetProcessHeap(), 0, pattern->info ); +} + BOOL get_brush_bitmap_info( HBRUSH handle, BITMAPINFO *info, void **bits, UINT *usage ) { BRUSHOBJ *brush; @@ -147,23 +196,23 @@ BOOL get_brush_bitmap_info( HBRUSH handle, BITMAPINFO *info, void **bits, UINT * if (!(brush = GDI_GetObjPtr( handle, OBJ_BRUSH ))) return FALSE; - if (!brush->info) + if (!brush->pattern.info) { - BITMAPOBJ *bmp = GDI_GetObjPtr( brush->bitmap, OBJ_BITMAP ); + BITMAPOBJ *bmp = GDI_GetObjPtr( brush->pattern.bitmap, OBJ_BITMAP ); if (bmp) { - store_bitmap_bits( brush, bmp ); - GDI_ReleaseObj( brush->bitmap ); + store_bitmap_bits( &brush->pattern, bmp ); + GDI_ReleaseObj( brush->pattern.bitmap ); } } - if (brush->info) + if (brush->pattern.info) { - memcpy( info, brush->info, get_dib_info_size( brush->info, brush->usage )); + memcpy( info, brush->pattern.info, get_dib_info_size( brush->pattern.info, brush->pattern.usage )); if (info->bmiHeader.biBitCount <= 8 && !info->bmiHeader.biClrUsed) fill_default_color_table( info ); - *bits = brush->bits.ptr; - *usage = brush->usage; + *bits = brush->pattern.bits.ptr; + *usage = brush->pattern.usage; ret = TRUE; } GDI_ReleaseObj( handle ); @@ -194,58 +243,19 @@ HBRUSH WINAPI CreateBrushIndirect( const LOGBRUSH * brush ) { BRUSHOBJ * ptr; HBRUSH hbrush; - HGLOBAL hmem = 0; - if (!(ptr = HeapAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*ptr) ))) return 0; + if (!(ptr = HeapAlloc( GetProcessHeap(), 0, sizeof(*ptr) ))) return 0; ptr->logbrush = *brush; - switch (ptr->logbrush.lbStyle) - { - case BS_SOLID: - case BS_HOLLOW: - case BS_HATCHED: - break; - - case BS_PATTERN8X8: - ptr->logbrush.lbStyle = BS_PATTERN; - /* fall through */ - case BS_PATTERN: - if (!copy_bitmap( ptr, (HBITMAP)ptr->logbrush.lbHatch )) goto error; - ptr->logbrush.lbColor = 0; - break; - - case BS_DIBPATTERN: - hmem = (HGLOBAL)ptr->logbrush.lbHatch; - if (!(ptr->logbrush.lbHatch = (ULONG_PTR)GlobalLock( hmem ))) goto error; - /* fall through */ - case BS_DIBPATTERNPT: - ptr->usage = ptr->logbrush.lbColor; - ptr->info = copy_packed_dib( (BITMAPINFO *)ptr->logbrush.lbHatch, ptr->usage ); - if (hmem) GlobalUnlock( hmem ); - if (!ptr->info) goto error; - ptr->bits.ptr = (char *)ptr->info + get_dib_info_size( ptr->info, ptr->usage ); - ptr->logbrush.lbStyle = BS_DIBPATTERN; - ptr->logbrush.lbColor = 0; - break; - - case BS_DIBPATTERN8X8: - case BS_MONOPATTERN: - case BS_INDEXED: - default: - WARN( "invalid brush style %u\n", ptr->logbrush.lbStyle ); - goto error; - } - - if ((hbrush = alloc_gdi_handle( &ptr->header, OBJ_BRUSH, &brush_funcs ))) + if (store_brush_pattern( &ptr->logbrush, &ptr->pattern ) && + (hbrush = alloc_gdi_handle( &ptr->header, OBJ_BRUSH, &brush_funcs ))) { TRACE("%p\n", hbrush); return hbrush; } - error: - if (ptr->bitmap) DeleteObject( ptr->bitmap ); - HeapFree( GetProcessHeap(), 0, ptr->info ); + free_brush_pattern( &ptr->pattern ); HeapFree( GetProcessHeap(), 0, ptr ); return 0; } @@ -476,22 +486,22 @@ static HGDIOBJ BRUSH_SelectObject( HGDIOBJ handle, HDC hdc ) if ((brush = GDI_GetObjPtr( handle, OBJ_BRUSH ))) { PHYSDEV physdev = GET_DC_PHYSDEV( dc, pSelectBrush ); - HBITMAP bitmap = brush->bitmap; + HBITMAP bitmap = brush->pattern.bitmap; BITMAPINFO *info; void *bits; UINT usage; - if (bitmap && !brush->info) + if (bitmap && !brush->pattern.info) { BITMAPOBJ *bmp = GDI_GetObjPtr( bitmap, OBJ_BITMAP ); /* fetch the bitmap bits if we are selecting into a different type of DC */ - if (bmp && bmp->funcs != physdev->funcs) store_bitmap_bits( brush, bmp ); + if (bmp && bmp->funcs != physdev->funcs) store_bitmap_bits( &brush->pattern, bmp ); GDI_ReleaseObj( bitmap ); } - info = brush->info; - bits = brush->bits.ptr; - usage = brush->usage; + info = brush->pattern.info; + bits = brush->pattern.bits.ptr; + usage = brush->pattern.usage; GDI_inc_ref_count( handle ); GDI_ReleaseObj( handle ); @@ -519,9 +529,7 @@ static BOOL BRUSH_DeleteObject( HGDIOBJ handle ) BRUSHOBJ *brush = free_gdi_handle( handle ); if (!brush) return FALSE; - if (brush->bits.free) brush->bits.free( &brush->bits ); - if (brush->bitmap) DeleteObject( brush->bitmap ); - HeapFree( GetProcessHeap(), 0, brush->info ); + free_brush_pattern( &brush->pattern ); return HeapFree( GetProcessHeap(), 0, brush ); } diff --git a/dlls/gdi32/gdi_private.h b/dlls/gdi32/gdi_private.h index c9822fe..daf3447 100644 --- a/dlls/gdi32/gdi_private.h +++ b/dlls/gdi32/gdi_private.h @@ -182,6 +182,14 @@ typedef struct tagBITMAPOBJ RGBQUAD *color_table; /* DIB color table if <= 8bpp (always 1 << bpp in size) */ } BITMAPOBJ; +struct brush_pattern +{ + HBITMAP bitmap; /* bitmap handle for DDB patterns */ + BITMAPINFO *info; /* DIB info */ + struct gdi_image_bits bits; /* DIB bits */ + UINT usage; /* color usage for DIB info */ +}; + /* bidi.c */ /* Wine_GCPW Flags */ @@ -208,6 +216,8 @@ extern DWORD stretch_bits( const BITMAPINFO *src_info, struct bitblt_coords *src struct gdi_image_bits *bits, int mode ) DECLSPEC_HIDDEN; /* brush.c */ +extern BOOL store_brush_pattern( LOGBRUSH *brush, struct brush_pattern *pattern ) DECLSPEC_HIDDEN; +extern void free_brush_pattern( struct brush_pattern *pattern ) DECLSPEC_HIDDEN; extern BOOL get_brush_bitmap_info( HBRUSH handle, BITMAPINFO *info, void **bits, UINT *usage ) DECLSPEC_HIDDEN; /* clipping.c */

1 0