August 2011 - Wine-commits - list.winehq.org

Vincent Povirk : gdiplus: Fix vertical string alignment.
by Alexandre Julliard 02 Aug '11

02 Aug '11

Module: wine Branch: master Commit: 380abdeb10119a3ae2210544da99a33cc54798bc URL: http://source.winehq.org/git/wine.git/?a=commit;h=380abdeb10119a3ae2210544d… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Tue Aug 2 11:19:16 2011 -0500 gdiplus: Fix vertical string alignment. --- dlls/gdiplus/graphics.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index 8ad0c1d..7fcff02 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -4643,7 +4643,8 @@ GpStatus WINGDIPAPI GdipDrawString(GpGraphics *graphics, GDIPCONST WCHAR *string GpPointF pt[3], rectcpy[4]; POINT corners[4]; REAL rel_width, rel_height; - INT offsety = 0, save_state; + INT save_state; + REAL offsety = 0.0; struct draw_string_args args; RectF scaled_rect; HDC hdc, temp_hdc=NULL; @@ -4719,7 +4720,7 @@ GpStatus WINGDIPAPI GdipDrawString(GpGraphics *graphics, GDIPCONST WCHAR *string args.brush = brush; args.x = rect->X; - args.y = rect->Y; + args.y = rect->Y + offsety; args.rel_width = rel_width; args.rel_height = rel_height;

1 0

Jay Yang : shell32/tests: Add tests for SHCreateDefaultContextMenu.
by Alexandre Julliard 02 Aug '11

02 Aug '11

Module: wine Branch: master Commit: 23c8704cb50818a0cdb586e9ae43e391e52c23c3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=23c8704cb50818a0cdb586e9a… Author: Jay Yang <jkelleyy(a)gmail.com> Date: Mon Aug 1 17:47:52 2011 -0400 shell32/tests: Add tests for SHCreateDefaultContextMenu. --- dlls/shell32/tests/shlfolder.c | 90 ++++++++++++++++++++++++++++++++++++++++ 1 files changed, 90 insertions(+), 0 deletions(-) diff --git a/dlls/shell32/tests/shlfolder.c b/dlls/shell32/tests/shlfolder.c index 0006155..c0533cb 100644 --- a/dlls/shell32/tests/shlfolder.c +++ b/dlls/shell32/tests/shlfolder.c @@ -71,6 +71,7 @@ static HRESULT (WINAPI *pSHGetIDListFromObject)(IUnknown*, PIDLIST_ABSOLUTE*); static HRESULT (WINAPI *pSHGetItemFromObject)(IUnknown*,REFIID,void**); static BOOL (WINAPI *pIsWow64Process)(HANDLE, PBOOL); static UINT (WINAPI *pGetSystemWow64DirectoryW)(LPWSTR, UINT); +static HRESULT (WINAPI *pSHCreateDefaultContextMenu)(const DEFCONTEXTMENU*,REFIID,void**); static WCHAR *make_wstr(const char *str) { @@ -126,6 +127,7 @@ static void init_function_pointers(void) MAKEFUNC(SHGetItemFromDataObject); MAKEFUNC(SHGetIDListFromObject); MAKEFUNC(SHGetItemFromObject); + MAKEFUNC(SHCreateDefaultContextMenu); #undef MAKEFUNC #define MAKEFUNC_ORD(f, ord) (p##f = (void*)GetProcAddress(hmod, (LPSTR)(ord))) @@ -4475,6 +4477,93 @@ static void test_SHChangeNotify(void) ok(br == TRUE, "RemoveDirectory failed: %d\n", GetLastError()); } +static void test_SHCreateDefaultContextMenu(void) +{ + HKEY keys[16]; + WCHAR path[MAX_PATH]; + IShellFolder *desktop,*folder; + IPersistFolder2 *persist; + IContextMenu *cmenu; + LONG status; + LPITEMIDLIST pidlFolder, pidl_child, pidl; + DEFCONTEXTMENU cminfo; + HRESULT hr; + UINT i; + const WCHAR filename[] = + {'\\','t','e','s','t','d','i','r','\\','t','e','s','t','1','.','t','x','t',0}; + if(!pSHCreateDefaultContextMenu) + { + todo_wine win_skip("SHCreateDefaultContextMenu missing.\n"); + return; + } + + if(!pSHBindToParent) + { + skip("SHBindToParent missing.\n"); + return; + } + + GetCurrentDirectoryW(MAX_PATH, path); + if(!lstrlenW(path)) + { + skip("GetCurrentDirectoryW returned an empty string.\n"); + return; + } + lstrcatW(path, filename); + SHGetDesktopFolder(&desktop); + + CreateFilesFolders(); + + hr = IShellFolder_ParseDisplayName(desktop, NULL, NULL, path, NULL, &pidl, 0); + ok(hr == S_OK || broken(hr == E_FAIL) /* WinME */, "Got 0x%08x\n", hr); + if(SUCCEEDED(hr)) + { + + hr = pSHBindToParent(pidl, &IID_IShellFolder, (void**)&folder, (LPCITEMIDLIST*)&pidl_child); + ok(hr == S_OK, "Got 0x%08x\n", hr); + + IShellFolder_QueryInterface(folder,&IID_IPersistFolder2,(void**)&persist); + IPersistFolder2_GetCurFolder(persist,&pidlFolder); + IPersistFolder2_Release(persist); + if(SUCCEEDED(hr)) + { + + cminfo.hwnd=NULL; + cminfo.pcmcb=NULL; + cminfo.psf=folder; + cminfo.pidlFolder=NULL; + cminfo.apidl=(LPCITEMIDLIST*)&pidl_child; + cminfo.cidl=1; + cminfo.aKeys=NULL; + cminfo.cKeys=0; + cminfo.punkAssociationInfo=NULL; + hr = pSHCreateDefaultContextMenu(&cminfo,&IID_IContextMenu,(void**)&cmenu); + ok(hr==S_OK,"Got 0x%08x\n", hr); + IContextMenu_Release(cmenu); + cminfo.pidlFolder=pidlFolder; + hr = pSHCreateDefaultContextMenu(&cminfo,&IID_IContextMenu,(void**)&cmenu); + ok(hr==S_OK,"Got 0x%08x\n", hr); + IContextMenu_Release(cmenu); + status = RegOpenKeyExA(HKEY_CLASSES_ROOT,"*",0,KEY_READ,keys); + if(status==ERROR_SUCCESS){ + for(i=1;i<16;i++) + keys[i]=keys[0]; + cminfo.aKeys=keys; + cminfo.cKeys=16; + hr = pSHCreateDefaultContextMenu(&cminfo,&IID_IContextMenu,(void**)&cmenu); + RegCloseKey(keys[0]); + ok(hr==S_OK,"Got 0x%08x\n", hr); + IContextMenu_Release(cmenu); + } + } + ILFree(pidlFolder); + IShellFolder_Release(folder); + } + IShellFolder_Release(desktop); + ILFree(pidl); + Cleanup(); +} + START_TEST(shlfolder) { init_function_pointers(); @@ -4509,6 +4598,7 @@ START_TEST(shlfolder) test_SHChangeNotify(); test_ShellItemBindToHandler(); test_ShellItemGetAttributes(); + test_SHCreateDefaultContextMenu(); OleUninitialize(); }

1 0

Camillo Lugaresi : configure: Fix problem with linking on OS X 10.7.
by Alexandre Julliard 02 Aug '11

02 Aug '11

Module: wine Branch: master Commit: 0b5dcbafea44bf68ec72494cc9f2c6f2dbb2cd16 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0b5dcbafea44bf68ec72494cc… Author: Camillo Lugaresi <camillo.lugaresi(a)gmail.com> Date: Tue Aug 2 04:31:53 2011 +0200 configure: Fix problem with linking on OS X 10.7. Some of the new link commands in the latest ld appear to be buggy and cause bus error while initializing libSystem. Fixed by reverting to 10.6 linking behavior. --- configure | 10 ++++++++-- configure.ac | 7 ++++++- 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 0fbfd51..3876127 100755 --- a/configure +++ b/configure @@ -6417,8 +6417,14 @@ fi APPLICATIONSERVICESLIB="-framework ApplicationServices" - LDEXECFLAGS="-image_base 0x7bf00000 -Wl,-segaddr,WINE_DOS,0x00001000,-segaddr,WINE_SHAREDHEAP,0x7f000000" - + case $host_os in + darwin11*) + LDEXECFLAGS="-image_base 0x7bf00000 -Wl,-macosx_version_min,10.6,-segaddr,WINE_DOS,0x00001000,-segaddr,WINE_SHAREDHEAP,0x7f000000" + ;; + *) + LDEXECFLAGS="-image_base 0x7bf00000 -Wl,-segaddr,WINE_DOS,0x00001000,-segaddr,WINE_SHAREDHEAP,0x7f000000" + ;; + esac if test "$ac_cv_header_DiskArbitration_DiskArbitration_h" = "yes" then DISKARBITRATIONLIB="-framework DiskArbitration -framework CoreFoundation" diff --git a/configure.ac b/configure.ac index 8104f7f..ecd74d6 100644 --- a/configure.ac +++ b/configure.ac @@ -697,7 +697,12 @@ case $host_os in AC_SUBST(COREFOUNDATIONLIB,"-framework CoreFoundation") AC_SUBST(IOKITLIB,"-framework IOKit -framework CoreFoundation") AC_SUBST(APPLICATIONSERVICESLIB,"-framework ApplicationServices") - AC_SUBST(LDEXECFLAGS,["-image_base 0x7bf00000 -Wl,-segaddr,WINE_DOS,0x00001000,-segaddr,WINE_SHAREDHEAP,0x7f000000"]) + case $host_os in + darwin11*) + AC_SUBST(LDEXECFLAGS,["-image_base 0x7bf00000 -Wl,-macosx_version_min,10.6,-segaddr,WINE_DOS,0x00001000,-segaddr,WINE_SHAREDHEAP,0x7f000000"]) ;; + *) + AC_SUBST(LDEXECFLAGS,["-image_base 0x7bf00000 -Wl,-segaddr,WINE_DOS,0x00001000,-segaddr,WINE_SHAREDHEAP,0x7f000000"]) ;; + esac if test "$ac_cv_header_DiskArbitration_DiskArbitration_h" = "yes" then dnl DiskArbitration API is not public on Darwin < 8.0, use it only if header found

1 0

Frédéric Delanoy : cmd/tests: Additional tests for variable delayed expansion.
by Alexandre Julliard 02 Aug '11

02 Aug '11

Module: wine Branch: master Commit: 0cfe9fa6a5e4ebb95394cd5b21a6d7a2bd9009b0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0cfe9fa6a5e4ebb95394cd5b2… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Tue Aug 2 10:10:52 2011 +0200 cmd/tests: Additional tests for variable delayed expansion. --- programs/cmd/tests/test_builtins.cmd | 9 +++++++++ programs/cmd/tests/test_builtins.cmd.exp | 5 +++++ 2 files changed, 14 insertions(+), 0 deletions(-) diff --git a/programs/cmd/tests/test_builtins.cmd b/programs/cmd/tests/test_builtins.cmd index 3f8c72e..eb79fb2 100644 --- a/programs/cmd/tests/test_builtins.cmd +++ b/programs/cmd/tests/test_builtins.cmd @@ -207,6 +207,15 @@ set FOO=foo echo %FOO% echo !FOO! set FOO= +echo ...using /V cmd flag +echo @echo off> tmp.cmd +echo set FOO=foo>> tmp.cmd +echo echo %%FOO%%>> tmp.cmd +echo echo !FOO!>> tmp.cmd +echo set FOO=>> tmp.cmd +cmd /V:ON /C tmp.cmd +cmd /V:OfF /C tmp.cmd +del tmp.cmd echo ------------ Testing conditional execution -------------- echo ...unconditional ^& diff --git a/programs/cmd/tests/test_builtins.cmd.exp b/programs/cmd/tests/test_builtins.cmd.exp index b754faf..463b225 100644 --- a/programs/cmd/tests/test_builtins.cmd.exp +++ b/programs/cmd/tests/test_builtins.cmd.exp @@ -161,6 +161,11 @@ foo 0(a)or_broken@1 foo !FOO! +...using /V cmd flag +foo +(a)todo_wine@foo(a)or_broken@!FOO! +foo +!FOO! ------------ Testing conditional execution -------------- @todo_wine(a)...unconditional & foo1

1 0

Frédéric Delanoy : dlls: Assorted spelling fixes.
by Alexandre Julliard 02 Aug '11

02 Aug '11

Module: wine Branch: master Commit: b5a33dab886df6352011457932713ed321976bf6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b5a33dab886df635201145793… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Wed Jul 27 13:38:43 2011 +0200 dlls: Assorted spelling fixes. --- dlls/dmloader/container.c | 12 ++++++------ dlls/dmloader/loaderstream.c | 2 +- dlls/dmscript/script.c | 8 ++++---- dlls/dmstyle/chordtrack.c | 2 +- dlls/dmstyle/dmutils.c | 2 +- dlls/dmstyle/style.c | 26 +++++++++++++------------- dlls/dmstyle/styletrack.c | 4 ++-- dlls/dmusic/collection.c | 12 ++++++------ dlls/dmusic/instrument.c | 10 +++++----- dlls/dplayx/dplayx_main.c | 4 ++-- dlls/dsound/buffer.c | 2 +- dlls/dswave/dswave.c | 8 ++++---- dlls/dxerr8/errors.dat | 18 +++++++++--------- dlls/dxerr8/errors.h | 18 +++++++++--------- dlls/dxerr9/errors.dat | 18 +++++++++--------- dlls/dxerr9/errors.h | 18 +++++++++--------- dlls/gameux/gamestatistics.c | 2 +- dlls/gameux/gameux_private.h | 2 +- dlls/gdi32/brush.c | 4 ++-- dlls/gdi32/freetype.c | 2 +- dlls/gdi32/gdi_private.h | 2 +- dlls/gdi32/region.c | 2 +- dlls/gdiplus/image.c | 2 +- dlls/ieframe/navigate.c | 4 ++-- dlls/imaadp32.acm/imaadp32.c | 4 ++-- dlls/imagehlp/integrity.c | 2 +- dlls/jscript/regexp.c | 2 +- dlls/kernel32/console.c | 4 ++-- dlls/kernel32/debugger.c | 2 +- dlls/kernel32/lcformat.c | 2 +- dlls/kernel32/locale.c | 2 +- dlls/kernel32/lzexpand.c | 2 +- dlls/kernel32/path.c | 2 +- dlls/krnl386.exe16/dosaspi.c | 2 +- dlls/krnl386.exe16/ioports.c | 6 +++--- dlls/krnl386.exe16/soundblaster.c | 2 +- dlls/mapi32/prop.c | 5 +++-- dlls/msacm32.drv/wavemap.c | 2 +- dlls/msacm32/pcmconverter.c | 4 ++-- dlls/msadp32.acm/msadp32.c | 4 ++-- 40 files changed, 116 insertions(+), 115 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=b5a33dab886df63520114…

1 0

Huw Davies : gdi32: Simplify the calculation of the rop masks.
by Alexandre Julliard 02 Aug '11

02 Aug '11

Module: wine Branch: master Commit: 4b50743f87e9c66faee0aaf059de392a28728a07 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4b50743f87e9c66faee0aaf05… Author: Huw Davies <huw(a)codeweavers.com> Date: Tue Aug 2 14:11:06 2011 +0100 gdi32: Simplify the calculation of the rop masks. --- dlls/gdi32/dibdrv/objects.c | 16 ++++++++-------- 1 files changed, 8 insertions(+), 8 deletions(-) diff --git a/dlls/gdi32/dibdrv/objects.c b/dlls/gdi32/dibdrv/objects.c index 0dcf0da..70e2904 100644 --- a/dlls/gdi32/dibdrv/objects.c +++ b/dlls/gdi32/dibdrv/objects.c @@ -57,11 +57,11 @@ WINE_DEFAULT_DEBUG_CHANNEL(dib); * */ -/* A = (P & A1) | (~P & A2) */ -#define ZERO {0, 0} -#define ONE {0xffffffff, 0xffffffff} -#define P {0xffffffff, 0} -#define NOT_P {0, 0xffffffff} +/* A = (P & A1) ^ A2 */ +#define ZERO { 0u, 0u} +#define ONE { 0u, ~0u} +#define P {~0u, 0u} +#define NOT_P {~0u, ~0u} static const DWORD rop2_and_array[16][2] = { @@ -71,7 +71,7 @@ static const DWORD rop2_and_array[16][2] = ZERO, NOT_P, NOT_P, ZERO }; -/* X = (P & X1) | (~P & X2) */ +/* X = (P & X1) ^ X2 */ static const DWORD rop2_xor_array[16][2] = { ZERO, NOT_P, ZERO, NOT_P, @@ -88,8 +88,8 @@ static const DWORD rop2_xor_array[16][2] = void calc_and_xor_masks(INT rop, DWORD color, DWORD *and, DWORD *xor) { /* NB The ROP2 codes start at one and the arrays are zero-based */ - *and = (color & rop2_and_array[rop-1][0]) | ((~color) & rop2_and_array[rop-1][1]); - *xor = (color & rop2_xor_array[rop-1][0]) | ((~color) & rop2_xor_array[rop-1][1]); + *and = (color & rop2_and_array[rop-1][0]) ^ rop2_and_array[rop-1][1]; + *xor = (color & rop2_xor_array[rop-1][0]) ^ rop2_xor_array[rop-1][1]; } static inline RGBQUAD rgbquad_from_colorref(COLORREF c)

1 0

Huw Davies : gdi32: Add a function to retrieve the rop codes.
by Alexandre Julliard 02 Aug '11

02 Aug '11

Module: wine Branch: master Commit: 76b0626ff2d251122c01fb609c2ed9f9b6842e60 URL: http://source.winehq.org/git/wine.git/?a=commit;h=76b0626ff2d251122c01fb609… Author: Huw Davies <huw(a)codeweavers.com> Date: Tue Aug 2 14:11:07 2011 +0100 gdi32: Add a function to retrieve the rop codes. --- dlls/gdi32/dibdrv/dibdrv.h | 6 ++++++ dlls/gdi32/dibdrv/objects.c | 17 ++++++++++++++--- 2 files changed, 20 insertions(+), 3 deletions(-) diff --git a/dlls/gdi32/dibdrv/dibdrv.h b/dlls/gdi32/dibdrv/dibdrv.h index 7889ebb..ee08bbd 100644 --- a/dlls/gdi32/dibdrv/dibdrv.h +++ b/dlls/gdi32/dibdrv/dibdrv.h @@ -56,6 +56,12 @@ extern const primitive_funcs funcs_4 DECLSPEC_HIDDEN; extern const primitive_funcs funcs_1 DECLSPEC_HIDDEN; extern const primitive_funcs funcs_null DECLSPEC_HIDDEN; +struct rop_codes +{ + DWORD a1, a2, x1, x2; +}; + +extern void get_rop_codes(INT rop, struct rop_codes *codes); extern void calc_and_xor_masks(INT rop, DWORD color, DWORD *and, DWORD *xor) DECLSPEC_HIDDEN; extern void update_brush_rop( dibdrv_physdev *pdev, INT rop ) DECLSPEC_HIDDEN; extern void reset_dash_origin(dibdrv_physdev *pdev) DECLSPEC_HIDDEN; diff --git a/dlls/gdi32/dibdrv/objects.c b/dlls/gdi32/dibdrv/objects.c index 70e2904..15ce72f 100644 --- a/dlls/gdi32/dibdrv/objects.c +++ b/dlls/gdi32/dibdrv/objects.c @@ -85,11 +85,22 @@ static const DWORD rop2_xor_array[16][2] = #undef ONE #undef ZERO -void calc_and_xor_masks(INT rop, DWORD color, DWORD *and, DWORD *xor) +void get_rop_codes(INT rop, struct rop_codes *codes) { /* NB The ROP2 codes start at one and the arrays are zero-based */ - *and = (color & rop2_and_array[rop-1][0]) ^ rop2_and_array[rop-1][1]; - *xor = (color & rop2_xor_array[rop-1][0]) ^ rop2_xor_array[rop-1][1]; + codes->a1 = rop2_and_array[rop-1][0]; + codes->a2 = rop2_and_array[rop-1][1]; + codes->x1 = rop2_xor_array[rop-1][0]; + codes->x2 = rop2_xor_array[rop-1][1]; +} + +void calc_and_xor_masks(INT rop, DWORD color, DWORD *and, DWORD *xor) +{ + struct rop_codes codes; + get_rop_codes( rop, &codes ); + + *and = (color & codes.a1) ^ codes.a2; + *xor = (color & codes.x1) ^ codes.x2; } static inline RGBQUAD rgbquad_from_colorref(COLORREF c)

1 0

Huw Davies : gdi32: Add a copy_rect primitive.
by Alexandre Julliard 02 Aug '11

02 Aug '11

Module: wine Branch: master Commit: 2521bf3cc2d236169921c7217df56cf3e970cd56 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2521bf3cc2d236169921c7217… Author: Huw Davies <huw(a)codeweavers.com> Date: Tue Aug 2 14:11:08 2011 +0100 gdi32: Add a copy_rect primitive. --- dlls/gdi32/dibdrv/dibdrv.h | 1 + dlls/gdi32/dibdrv/primitives.c | 331 ++++++++++++++++++++++++++++++++++++++++ 2 files changed, 332 insertions(+), 0 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=2521bf3cc2d236169921c…

1 0

Huw Davies : gdi32: Move the colour setting to a separate function.
by Alexandre Julliard 02 Aug '11

02 Aug '11

Module: wine Branch: master Commit: 1da138ced0cf628868ae0aaf00cde05c1af59ceb URL: http://source.winehq.org/git/wine.git/?a=commit;h=1da138ced0cf628868ae0aaf0… Author: Huw Davies <huw(a)codeweavers.com> Date: Tue Aug 2 14:11:09 2011 +0100 gdi32: Move the colour setting to a separate function. --- dlls/gdi32/dibdrv/dc.c | 75 +++++++++++++++++++++++++++--------------------- 1 files changed, 42 insertions(+), 33 deletions(-) diff --git a/dlls/gdi32/dibdrv/dc.c b/dlls/gdi32/dibdrv/dc.c index aa74859..dacd40b 100644 --- a/dlls/gdi32/dibdrv/dc.c +++ b/dlls/gdi32/dibdrv/dc.c @@ -306,6 +306,46 @@ static BOOL dibdrv_DeleteDC( PHYSDEV dev ) return 0; } +static void set_color_info( const dib_info *dib, BITMAPINFO *info ) +{ + info->bmiHeader.biCompression = BI_RGB; + info->bmiHeader.biClrUsed = 0; + + switch (info->bmiHeader.biBitCount) + { + case 1: + case 4: + case 8: + if (dib->color_table) + { + info->bmiHeader.biClrUsed = min( dib->color_table_size, 1 << dib->bit_count ); + memcpy( info->bmiColors, dib->color_table, + info->bmiHeader.biClrUsed * sizeof(RGBQUAD) ); + } + break; + case 16: + if (dib->funcs != &funcs_555) + { + DWORD *masks = (DWORD *)info->bmiColors; + masks[0] = dib->red_mask; + masks[1] = dib->green_mask; + masks[2] = dib->blue_mask; + info->bmiHeader.biCompression = BI_BITFIELDS; + } + break; + case 32: + if (dib->funcs != &funcs_8888) + { + DWORD *masks = (DWORD *)info->bmiColors; + masks[0] = dib->red_mask; + masks[1] = dib->green_mask; + masks[2] = dib->blue_mask; + info->bmiHeader.biCompression = BI_BITFIELDS; + } + break; + } +} + /*********************************************************************** * dibdrv_GetImage */ @@ -379,39 +419,8 @@ static DWORD dibdrv_GetImage( PHYSDEV dev, HBITMAP hbitmap, BITMAPINFO *info, info->bmiHeader.biBitCount = pdev->dib.bit_count; info->bmiHeader.biSizeImage = pdev->dib.height * abs(pdev->dib.stride); - switch (info->bmiHeader.biBitCount) - { - case 1: - case 4: - case 8: - if (pdev->dib.color_table) - { - info->bmiHeader.biClrUsed = min( pdev->dib.color_table_size, 1 << pdev->dib.bit_count ); - memcpy( info->bmiColors, pdev->dib.color_table, - info->bmiHeader.biClrUsed * sizeof(RGBQUAD) ); - } - break; - case 16: - if (pdev->dib.funcs != &funcs_555) - { - DWORD *masks = (DWORD *)info->bmiColors; - masks[0] = pdev->dib.red_mask; - masks[1] = pdev->dib.green_mask; - masks[2] = pdev->dib.blue_mask; - info->bmiHeader.biCompression = BI_BITFIELDS; - } - break; - case 32: - if (pdev->dib.funcs != &funcs_8888) - { - DWORD *masks = (DWORD *)info->bmiColors; - masks[0] = pdev->dib.red_mask; - masks[1] = pdev->dib.green_mask; - masks[2] = pdev->dib.blue_mask; - info->bmiHeader.biCompression = BI_BITFIELDS; - } - break; - } + set_color_info( &pdev->dib, info ); + if (bits) { bits->ptr = pdev->dib.bits;

1 0

Huw Davies : gdi32: Add a PutImage implementation to the dib driver.
by Alexandre Julliard 02 Aug '11

02 Aug '11

Module: wine Branch: master Commit: d4076d438cb5ff05f6800cdaae7ac88844e70595 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d4076d438cb5ff05f6800cdaa… Author: Huw Davies <huw(a)codeweavers.com> Date: Tue Aug 2 14:11:10 2011 +0100 gdi32: Add a PutImage implementation to the dib driver. --- dlls/gdi32/dibdrv/dc.c | 187 +++++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 186 insertions(+), 1 deletions(-) diff --git a/dlls/gdi32/dibdrv/dc.c b/dlls/gdi32/dibdrv/dc.c index dacd40b..d9a2443 100644 --- a/dlls/gdi32/dibdrv/dc.c +++ b/dlls/gdi32/dibdrv/dc.c @@ -293,6 +293,33 @@ static void update_masks( dibdrv_physdev *pdev, INT rop ) calc_and_xor_masks( rop, pdev->bkgnd_color, &pdev->bkgnd_and, &pdev->bkgnd_xor ); } + /*********************************************************************** + * add_extra_clipping_region + * + * Temporarily add a region to the current clipping region. + * The returned region must be restored with restore_clipping_region. + */ +static HRGN add_extra_clipping_region( dibdrv_physdev *pdev, HRGN rgn ) +{ + HRGN ret, clip; + + if (!(clip = CreateRectRgn( 0, 0, 0, 0 ))) return 0; + CombineRgn( clip, pdev->clip, rgn, RGN_AND ); + ret = pdev->clip; + pdev->clip = clip; + return ret; +} + +/*********************************************************************** + * restore_clipping_region + */ +static void restore_clipping_region( dibdrv_physdev *pdev, HRGN rgn ) +{ + if (!rgn) return; + DeleteObject( pdev->clip ); + pdev->clip = rgn; +} + /*********************************************************************** * dibdrv_DeleteDC */ @@ -433,6 +460,164 @@ static DWORD dibdrv_GetImage( PHYSDEV dev, HBITMAP hbitmap, BITMAPINFO *info, return ERROR_SUCCESS; } +static BOOL matching_color_info( const dib_info *dib, const BITMAPINFO *info ) +{ + switch (info->bmiHeader.biBitCount) + { + case 1: + case 4: + case 8: + { + RGBQUAD *color_table = (RGBQUAD *)((char *)info + info->bmiHeader.biSize); + if (dib->color_table_size != info->bmiHeader.biClrUsed ) return FALSE; + return memcmp( color_table, dib->color_table, dib->color_table_size * sizeof(RGBQUAD) ); + } + + case 16: + { + DWORD *masks = (DWORD *)info->bmiColors; + if (info->bmiHeader.biCompression == BI_RGB) return dib->funcs == &funcs_555; + if (info->bmiHeader.biCompression == BI_BITFIELDS) + return masks[0] == dib->red_mask && masks[1] == dib->green_mask && masks[2] == dib->blue_mask; + break; + } + + case 24: + return TRUE; + + case 32: + { + DWORD *masks = (DWORD *)info->bmiColors; + if (info->bmiHeader.biCompression == BI_RGB) return dib->funcs == &funcs_8888; + if (info->bmiHeader.biCompression == BI_BITFIELDS) + return masks[0] == dib->red_mask && masks[1] == dib->green_mask && masks[2] == dib->blue_mask; + break; + } + + } + + return FALSE; +} + +static inline BOOL rop_uses_pat(DWORD rop) +{ + return ((rop >> 4) & 0x0f0000) != (rop & 0x0f0000); +} + +/*********************************************************************** + * dibdrv_PutImage + */ +static DWORD dibdrv_PutImage( PHYSDEV dev, HBITMAP hbitmap, HRGN clip, BITMAPINFO *info, + const struct gdi_image_bits *bits, struct bitblt_coords *src, + struct bitblt_coords *dst, DWORD rop ) +{ + dib_info *dib, stand_alone; + DWORD ret; + RECT rect; + POINT origin; + dib_info src_dib; + HRGN total_clip, saved_clip = NULL; + dibdrv_physdev *pdev = NULL; + const WINEREGION *clip_data; + int i, rop2; + + TRACE( "%p %p %p\n", dev, hbitmap, info ); + + if (!hbitmap && rop_uses_pat( rop )) + { + PHYSDEV next = GET_NEXT_PHYSDEV( dev, pPutImage ); + FIXME( "rop %08x unsupported, forwarding to graphics driver\n", rop ); + return next->funcs->pPutImage( next, 0, clip, info, bits, src, dst, rop ); + } + + if (hbitmap) + { + BITMAPOBJ *bmp = GDI_GetObjPtr( hbitmap, OBJ_BITMAP ); + + if (!bmp) return ERROR_INVALID_HANDLE; + assert(bmp->dib); + + if (!init_dib_info( &stand_alone, &bmp->dib->dsBmih, bmp->dib->dsBitfields, + bmp->color_table, bmp->nb_colors, bmp->dib->dsBm.bmBits, 0 )) + { + ret = ERROR_BAD_FORMAT; + goto done; + } + dib = &stand_alone; + } + else + dib = &pdev->dib; + + if (info->bmiHeader.biPlanes != 1) goto update_format; + if (info->bmiHeader.biBitCount != dib->bit_count) goto update_format; + if (!matching_color_info( dib, info )) goto update_format; + if (!bits) + { + ret = ERROR_SUCCESS; + goto done; + } + if ((src->width != dst->width) || (src->height != dst->height)) + { + ret = ERROR_TRANSFORM_NOT_SUPPORTED; + goto done; + } + + init_dib_info_from_bitmapinfo( &src_dib, info, bits->ptr, 0 ); + + rect.left = dst->x; + rect.top = dst->y; + rect.right = dst->x + dst->width; + rect.bottom = dst->y + dst->height; + origin.x = src->x; + origin.y = src->y; + + if (hbitmap) + { + total_clip = clip; + rop2 = R2_COPYPEN; + } + else + { + if (clip) saved_clip = add_extra_clipping_region( pdev, clip ); + total_clip = pdev->clip; + rop2 = ((rop >> 16) & 0xf) + 1; + } + + if (total_clip == NULL) dib->funcs->copy_rect( dib, &rect, &src_dib, &origin, rop2 ); + else + { + clip_data = get_wine_region( total_clip ); + for (i = 0; i < clip_data->numRects; i++) + { + RECT clipped_rect; + + if (intersect_rect( &clipped_rect, &rect, clip_data->rects + i )) + { + origin.x = src->x + clipped_rect.left - dst->x; + origin.y = src->y + clipped_rect.top - dst->y; + dib->funcs->copy_rect( dib, &clipped_rect, &src_dib, &origin, rop2 ); + } + } + release_wine_region( total_clip ); + } + ret = ERROR_SUCCESS; + + if (saved_clip) restore_clipping_region( pdev, saved_clip ); + + goto done; + +update_format: + info->bmiHeader.biPlanes = 1; + info->bmiHeader.biBitCount = dib->bit_count; + set_color_info( dib, info ); + ret = ERROR_BAD_FORMAT; + +done: + if (hbitmap) GDI_ReleaseObj( hbitmap ); + + return ret; +} + /*********************************************************************** * dibdrv_SelectBitmap */ @@ -619,7 +804,7 @@ const DC_FUNCTIONS dib_driver = NULL, /* pPolygon */ NULL, /* pPolyline */ NULL, /* pPolylineTo */ - NULL, /* pPutImage */ + dibdrv_PutImage, /* pPutImage */ NULL, /* pRealizeDefaultPalette */ NULL, /* pRealizePalette */ dibdrv_Rectangle, /* pRectangle */

1 0