wine-commits March 2008

wine-commits@winehq.org

4 participants
1001 discussions

Stefan Dösinger : ddraw: Catch No-op SetRenderTarget operations.
by Alexandre Julliard 03 Mar '08

03 Mar '08

Module: wine Branch: master Commit: 069b9c8c1c639252a587590bfb78aebd59b6ea5a URL: http://source.winehq.org/git/wine.git/?a=commit;h=069b9c8c1c639252a587590bf… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Thu Feb 28 19:10:19 2008 +0100 ddraw: Catch No-op SetRenderTarget operations. --- dlls/ddraw/device.c | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/dlls/ddraw/device.c b/dlls/ddraw/device.c index b60b6f3..eed378b 100644 --- a/dlls/ddraw/device.c +++ b/dlls/ddraw/device.c @@ -1862,6 +1862,13 @@ IDirect3DDeviceImpl_7_SetRenderTarget(IDirect3DDevice7 *iface, EnterCriticalSection(&ddraw_cs); /* Flags: Not used */ + if(This->target == Target) + { + TRACE("No-op SetRenderTarget operation, not doing anything\n"); + LeaveCriticalSection(&ddraw_cs); + return D3D_OK; + } + hr = IWineD3DDevice_SetRenderTarget(This->wineD3DDevice, 0, Target ? Target->WineD3DSurface : NULL);

1 0

Stefan Dösinger : wined3d: Call fixup_extensions before destroying the fake context.
by Alexandre Julliard 03 Mar '08

03 Mar '08

Module: wine Branch: master Commit: 5709a0b4b6af7cd211a491de09a2837d03c7ba91 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5709a0b4b6af7cd211a491de0… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sun Feb 24 10:19:01 2008 +0100 wined3d: Call fixup_extensions before destroying the fake context. --- dlls/wined3d/directx.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c index 5b7333c..b7c56a6 100644 --- a/dlls/wined3d/directx.c +++ b/dlls/wined3d/directx.c @@ -3397,10 +3397,11 @@ BOOL InitAdapters(void) { TRACE("iPixelFormat=%d, RGBA=%d/%d/%d/%d, depth=%d, stencil=%d\n", cfgs->iPixelFormat, cfgs->redSize, cfgs->greenSize, cfgs->blueSize, cfgs->alphaSize, cfgs->depthSize, cfgs->stencilSize); cfgs++; } - WineD3D_ReleaseFakeGLContext(); fixup_extensions(&Adapters[0].gl_info); + WineD3D_ReleaseFakeGLContext(); + select_shader_mode(&Adapters[0].gl_info, WINED3DDEVTYPE_HAL, &ps_selected_mode, &vs_selected_mode); select_shader_max_constants(ps_selected_mode, vs_selected_mode, &Adapters[0].gl_info); fillGLAttribFuncs(&Adapters[0].gl_info);

1 0

Jacek Caban : wininet: Moved InternetReadFileExA to vtbl.
by Alexandre Julliard 03 Mar '08

03 Mar '08

Module: wine Branch: master Commit: d597fd12fc3b08ff69ef3fa2442e934b0f3c835d URL: http://source.winehq.org/git/wine.git/?a=commit;h=d597fd12fc3b08ff69ef3fa24… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Mar 3 18:07:20 2008 +0100 wininet: Moved InternetReadFileExA to vtbl. --- dlls/wininet/ftp.c | 3 + dlls/wininet/http.c | 69 ++++++++++++++++++++++ dlls/wininet/internet.c | 147 ++++------------------------------------------ dlls/wininet/internet.h | 1 + 4 files changed, 86 insertions(+), 134 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=d597fd12fc3b08ff69ef3…

1 0

Alexandre Julliard : Replace some instances of LoadStringA by LoadStringW.
by Alexandre Julliard 03 Mar '08

03 Mar '08

Module: wine Branch: master Commit: 4fd668d01088e82b4639ee2f4f088e0b718e6a88 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4fd668d01088e82b4639ee2f4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 3 20:07:37 2008 +0100 Replace some instances of LoadStringA by LoadStringW. --- dlls/comdlg32/printdlg.c | 21 +++++++++------------ dlls/oledlg/insobjdlg.c | 34 +++++++++++++++++----------------- dlls/sane.ds/ui.c | 22 +++++++++++----------- 3 files changed, 37 insertions(+), 40 deletions(-) diff --git a/dlls/comdlg32/printdlg.c b/dlls/comdlg32/printdlg.c index 73588b5..f2d67b5 100644 --- a/dlls/comdlg32/printdlg.c +++ b/dlls/comdlg32/printdlg.c @@ -308,15 +308,12 @@ static BOOL PRINTDLG_UpdatePrintDlgA(HWND hDlg, nToPage = GetDlgItemInt(hDlg, edt2, NULL, FALSE); if (nFromPage < lppd->nMinPage || nFromPage > lppd->nMaxPage || nToPage < lppd->nMinPage || nToPage > lppd->nMaxPage) { - char resourcestr[256]; - char resultstr[256]; - LoadStringA(COMDLG32_hInstance, PD32_INVALID_PAGE_RANGE, - resourcestr, 255); - sprintf(resultstr,resourcestr, lppd->nMinPage, lppd->nMaxPage); - LoadStringA(COMDLG32_hInstance, PD32_PRINT_TITLE, - resourcestr, 255); - MessageBoxA(hDlg, resultstr, resourcestr, - MB_OK | MB_ICONWARNING); + WCHAR resourcestr[256]; + WCHAR resultstr[256]; + LoadStringW(COMDLG32_hInstance, PD32_INVALID_PAGE_RANGE, resourcestr, 255); + wsprintfW(resultstr,resourcestr, lppd->nMinPage, lppd->nMaxPage); + LoadStringW(COMDLG32_hInstance, PD32_PRINT_TITLE, resourcestr, 255); + MessageBoxW(hDlg, resultstr, resourcestr, MB_OK | MB_ICONWARNING); return FALSE; } lppd->nFromPage = nFromPage; @@ -3569,9 +3566,9 @@ BOOL WINAPI PageSetupDlgA(LPPAGESETUPDLGA setupdlg) { bRet = PrintDlgA(&pdlg); if (!bRet){ if (!(setupdlg->Flags & PSD_NOWARNING)) { - char errstr[256]; - LoadStringA(COMDLG32_hInstance, PD32_NO_DEFAULT_PRINTER, errstr, 255); - MessageBoxA(setupdlg->hwndOwner, errstr, 0, MB_OK | MB_ICONERROR); + WCHAR errstr[256]; + LoadStringW(COMDLG32_hInstance, PD32_NO_DEFAULT_PRINTER, errstr, 255); + MessageBoxW(setupdlg->hwndOwner, errstr, 0, MB_OK | MB_ICONERROR); } return FALSE; } diff --git a/dlls/oledlg/insobjdlg.c b/dlls/oledlg/insobjdlg.c index 159b002..4578bbe 100644 --- a/dlls/oledlg/insobjdlg.c +++ b/dlls/oledlg/insobjdlg.c @@ -305,7 +305,7 @@ static void UIINSERTOBJECTDLG_SelectCreateNew(InsertObjectDlgInfo* pdlgInfo) */ static void UIINSERTOBJECTDLG_SelectCreateFromFile(InsertObjectDlgInfo* pdlgInfo) { - char resstr[MAX_PATH]; + WCHAR resstr[MAX_PATH]; ShowWindow(pdlgInfo->hwndAddCtrlBTN, SW_HIDE); ShowWindow(pdlgInfo->hwndObjTypeLBL, SW_HIDE); @@ -318,10 +318,10 @@ static void UIINSERTOBJECTDLG_SelectCreateFromFile(InsertObjectDlgInfo* pdlgInfo ShowWindow(pdlgInfo->hwndFileTB, SW_SHOW); ShowWindow(pdlgInfo->hwndBrowseBTN, SW_SHOW); - SendMessageA(pdlgInfo->hwndCreateFromFileCB, BM_SETCHECK, BST_CHECKED, 0); + SendMessageW(pdlgInfo->hwndCreateFromFileCB, BM_SETCHECK, BST_CHECKED, 0); - if (LoadStringA(OLEDLG_hInstance, IDS_RESULTFILEOBJDESC, resstr, MAX_PATH)) - SendMessageA(pdlgInfo->hwndResultDesc, WM_SETTEXT, 0, (LPARAM)resstr); + if (LoadStringW(OLEDLG_hInstance, IDS_RESULTFILEOBJDESC, resstr, MAX_PATH)) + SendMessageW(pdlgInfo->hwndResultDesc, WM_SETTEXT, 0, (LPARAM)resstr); } @@ -418,20 +418,20 @@ static void UIINSERTOBJECTDLG_FreeObjectTypes(InsertObjectDlgInfo* pdlgInfo) static void UIINSERTOBJECTDLG_SelChange(InsertObjectDlgInfo* pdlgInfo) { INT index; - CHAR objname[MAX_PATH]; - CHAR objdesc[MAX_PATH]; - CHAR resstr[MAX_PATH]; + WCHAR objname[MAX_PATH]; + WCHAR objdesc[MAX_PATH]; + WCHAR resstr[MAX_PATH]; TRACE("\n"); - if (LoadStringA(OLEDLG_hInstance, IDS_RESULTOBJDESC, resstr, MAX_PATH) && - ((index = SendMessageA(pdlgInfo->hwndObjTypeLB, LB_GETCURSEL, 0, 0)) >= 0) && - SendMessageA(pdlgInfo->hwndObjTypeLB, LB_GETTEXT, (WPARAM)index, (LPARAM)objname)) - sprintf(objdesc, resstr, objname); + if (LoadStringW(OLEDLG_hInstance, IDS_RESULTOBJDESC, resstr, MAX_PATH) && + ((index = SendMessageW(pdlgInfo->hwndObjTypeLB, LB_GETCURSEL, 0, 0)) >= 0) && + SendMessageW(pdlgInfo->hwndObjTypeLB, LB_GETTEXT, (WPARAM)index, (LPARAM)objname)) + wsprintfW(objdesc, resstr, objname); else objdesc[0] = 0; - SendMessageA(pdlgInfo->hwndResultDesc, WM_SETTEXT, 0, (LPARAM)objdesc); + SendMessageW(pdlgInfo->hwndResultDesc, WM_SETTEXT, 0, (LPARAM)objdesc); } @@ -615,13 +615,13 @@ static void UIINSERTOBJECTDLG_AddControl(InsertObjectDlgInfo* pdlgInfo) if (!bValid) { - char title[32]; - char msg[256]; + WCHAR title[32]; + WCHAR msg[256]; - LoadStringA(OLEDLG_hInstance, IDS_NOTOLEMODCAPTION, title, 32); - LoadStringA(OLEDLG_hInstance, IDS_NOTOLEMOD, msg, 256); + LoadStringW(OLEDLG_hInstance, IDS_NOTOLEMODCAPTION, title, 32); + LoadStringW(OLEDLG_hInstance, IDS_NOTOLEMOD, msg, 256); - MessageBoxA(pdlgInfo->hwndSelf, msg, title, MB_ICONEXCLAMATION); + MessageBoxW(pdlgInfo->hwndSelf, msg, title, MB_ICONEXCLAMATION); } } } diff --git a/dlls/sane.ds/ui.c b/dlls/sane.ds/ui.c index 6763240..83006c5 100644 --- a/dlls/sane.ds/ui.c +++ b/dlls/sane.ds/ui.c @@ -608,14 +608,13 @@ BOOL DoScannerUI(void) static void UpdateRelevantEdit(HWND hwnd, const SANE_Option_Descriptor *opt, int index, int position) { - CHAR buffer[244]; + WCHAR buffer[244]; HWND edit_w; - CHAR unit[20]; - - LoadStringA(SANE_instance, opt->unit, unit,20); + int len; if (opt->type == SANE_TYPE_INT) { + static const WCHAR formatW[] = {'%','i',0}; INT si; if (opt->constraint.range->quant) @@ -623,11 +622,11 @@ static void UpdateRelevantEdit(HWND hwnd, const SANE_Option_Descriptor *opt, else si = position; - sprintf(buffer,"%i %s",si,unit); - + len = wsprintfW( buffer, formatW, si ); } else if (opt->type == SANE_TYPE_FIXED) { + static const WCHAR formatW[] = {'%','f',0}; double s_quant, dd; s_quant = SANE_UNFIX(opt->constraint.range->quant); @@ -637,14 +636,15 @@ static void UpdateRelevantEdit(HWND hwnd, const SANE_Option_Descriptor *opt, else dd = position * 0.01; - sprintf(buffer,"%f %s",dd,unit); + len = wsprintfW( buffer, formatW, dd ); } - else - buffer[0] = 0; + else return; + + buffer[len++] = ' '; + LoadStringW( SANE_instance, opt->unit, buffer + len, sizeof(buffer)/sizeof(WCHAR) - len ); edit_w = GetDlgItem(hwnd,index+ID_BASE+ID_EDIT_BASE); - if (edit_w && buffer[0]) - SetWindowTextA(edit_w,buffer); + if (edit_w) SetWindowTextW(edit_w,buffer); }

1 0

Alexandre Julliard : user32: Reimplement LoadStringA to avoid memory allocations and to pass the tests.
by Alexandre Julliard 03 Mar '08

03 Mar '08

Module: wine Branch: master Commit: 28d29300748e0bdd1b89c686b97696aa696652ee URL: http://source.winehq.org/git/wine.git/?a=commit;h=28d29300748e0bdd1b89c686b… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 3 20:06:54 2008 +0100 user32: Reimplement LoadStringA to avoid memory allocations and to pass the tests. --- dlls/user32/resource.c | 36 ++++++++++++++++++------------------ dlls/user32/tests/resource.c | 7 ++----- 2 files changed, 20 insertions(+), 23 deletions(-) diff --git a/dlls/user32/resource.c b/dlls/user32/resource.c index b9a51ae..9eed0e8 100644 --- a/dlls/user32/resource.c +++ b/dlls/user32/resource.c @@ -24,6 +24,7 @@ #include "windef.h" #include "winbase.h" #include "winerror.h" +#include "winternl.h" #include "winnls.h" #include "wine/winbase16.h" #include "wine/winuser16.h" @@ -400,32 +401,31 @@ INT WINAPI LoadStringW( HINSTANCE instance, UINT resource_id, /********************************************************************** * LoadStringA (USER32.@) */ -INT WINAPI LoadStringA( HINSTANCE instance, UINT resource_id, - LPSTR buffer, INT buflen ) +INT WINAPI LoadStringA( HINSTANCE instance, UINT resource_id, LPSTR buffer, INT buflen ) { - INT retval; - LPWSTR wbuf; + HGLOBAL hmem; + HRSRC hrsrc; + DWORD retval = 0; TRACE("instance = %p, id = %04x, buffer = %p, length = %d\n", instance, resource_id, buffer, buflen); - if(buffer == NULL) /* asked size of string */ - return LoadStringW(instance, resource_id, NULL, 0); + if (!buflen) return -1; - wbuf = HeapAlloc(GetProcessHeap(), 0, buflen * sizeof(WCHAR)); - if(!wbuf) - return 0; - - retval = LoadStringW(instance, resource_id, wbuf, buflen); - if(retval != 0) + /* Use loword (incremented by 1) as resourceid */ + if ((hrsrc = FindResourceW( instance, MAKEINTRESOURCEW((LOWORD(resource_id) >> 4) + 1), + (LPWSTR)RT_STRING )) && + (hmem = LoadResource( instance, hrsrc ))) { - retval = WideCharToMultiByte(CP_ACP, 0, wbuf, retval, buffer, buflen - 1, NULL, NULL); - buffer[retval] = 0; - TRACE("%s loaded !\n", debugstr_a(buffer)); - } - else buffer[0] = 0; /* no check of buflen here */ - HeapFree( GetProcessHeap(), 0, wbuf ); + const WCHAR *p = LockResource(hmem); + unsigned int id = resource_id & 0x000f; + + while (id--) p += *p + 1; + RtlUnicodeToMultiByteN( buffer, buflen - 1, &retval, p + 1, *p * sizeof(WCHAR) ); + } + buffer[retval] = 0; + TRACE("returning %s\n", debugstr_a(buffer)); return retval; } diff --git a/dlls/user32/tests/resource.c b/dlls/user32/tests/resource.c index c2055dd..0fd77b7 100644 --- a/dlls/user32/tests/resource.c +++ b/dlls/user32/tests/resource.c @@ -120,11 +120,8 @@ static void test_LoadStringA (void) "LoadString failed: ret %d err %d\n", ret, GetLastError()); ret = LoadStringA(hInst, 0, buf, 0); - todo_wine - { - ok( ret == -1, "LoadStringA did not return -1 when called with buflen = 0, got %d, err %d\n", - ret, GetLastError()); - } + ok( ret == -1, "LoadStringA did not return -1 when called with buflen = 0, got %d, err %d\n", + ret, GetLastError()); } static void test_accel1(void)

1 0

Christopher Berner : user32/tests: Conformance tests for LoadStringW and LoadStringA.
by Alexandre Julliard 03 Mar '08

03 Mar '08

Module: wine Branch: master Commit: 4b130aa0b03b29abb8d576190dfab9c9cd324853 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4b130aa0b03b29abb8d576190… Author: Christopher Berner <raccoonone(a)procyongames.com> Date: Sun Mar 2 09:01:55 2008 -0800 user32/tests: Conformance tests for LoadStringW and LoadStringA. --- dlls/user32/tests/resource.c | 54 ++++++++++++++++++++++++++++++++++++++++- dlls/user32/tests/resource.rc | 1 + 2 files changed, 54 insertions(+), 1 deletions(-) diff --git a/dlls/user32/tests/resource.c b/dlls/user32/tests/resource.c index 59f86ef..c2055dd 100644 --- a/dlls/user32/tests/resource.c +++ b/dlls/user32/tests/resource.c @@ -31,6 +31,50 @@ static void init_function_pointers(void) pPrivateExtractIconsA = (void*)GetProcAddress(hmod, "PrivateExtractIconsA"); } +static void test_LoadStringW(void) +{ + HINSTANCE hInst = GetModuleHandle(NULL); + WCHAR copiedstringw[128], returnedstringw[128], *resourcepointer = NULL; + char copiedstring[128], returnedstring[128]; + int length1, length2, retvalue; + + /* Check that the string which is returned by LoadStringW matches + the string at the pointer returned by LoadStringW when called with buflen = 0 */ + length1 = LoadStringW(hInst, 2, (WCHAR *) &resourcepointer, 0); /* get pointer to resource. */ + length2 = LoadStringW(hInst, 2, returnedstringw, sizeof(returnedstringw) /sizeof(WCHAR)); /* get resource string */ + ok(length2 > 0, "LoadStringW failed to load resource 2, ret %d, err %d\n", length2, GetLastError()); + todo_wine + { + ok(length1 == length2, "LoadStringW returned different values dependent on buflen. ret1 %d, ret2 %d\n", + length1, length2); + ok(length1 > 0 && resourcepointer != NULL, "LoadStringW failed to get pointer to resource 2, ret %d, err %d\n", + length1, GetLastError()); + } + + /* Copy the resource since it is not '\0' terminated, and add '\0' to the end */ + if(resourcepointer != NULL) /* Check that the resource pointer was loaded to avoid access violation */ + { + memcpy(copiedstringw, resourcepointer, length1 * sizeof(WCHAR)); + copiedstringw[length1] = '\0'; + /* check that strings match */ + WideCharToMultiByte( CP_ACP, 0, returnedstringw, -1, returnedstring, 128, NULL, NULL ); + WideCharToMultiByte( CP_ACP, 0, copiedstringw, -1, copiedstring, 128, NULL, NULL ); + ok(!memcmp(copiedstringw, returnedstringw, (length2 + 1)*sizeof(WCHAR)), + "strings don't match: returnedstring = %s, copiedstring = %s\n", returnedstring, copiedstring); + } + todo_wine + { + /* check that calling LoadStringW with buffer = NULL returns zero */ + retvalue = LoadStringW(hInst, 2, NULL, 0); + ok(!retvalue, "LoadStringW returned a non-zero value when called with buffer = NULL, retvalue = %d", + retvalue); + /* check again, with a different buflen value, that calling LoadStringW with buffer = NULL returns zero */ + retvalue = LoadStringW(hInst, 2, NULL, 128); + ok(!retvalue, "LoadStringW returned a non-zero value when called with buffer = NULL, retvalue = %d", + retvalue); + } +} + static void test_LoadStringA (void) { HINSTANCE hInst = GetModuleHandle (NULL); @@ -74,6 +118,13 @@ static void test_LoadStringA (void) "LoadString failed: ret %d err %d\n", ret, GetLastError()); ok( LoadStringA( hInst, MAKELONG( 65534, 0xffff ), buf, sizeof(buf)) == ret, "LoadString failed: ret %d err %d\n", ret, GetLastError()); + + ret = LoadStringA(hInst, 0, buf, 0); + todo_wine + { + ok( ret == -1, "LoadStringA did not return -1 when called with buflen = 0, got %d, err %d\n", + ret, GetLastError()); + } } static void test_accel1(void) @@ -316,7 +367,8 @@ static void test_LoadImage(void) START_TEST(resource) { init_function_pointers(); - test_LoadStringA (); + test_LoadStringA(); + test_LoadStringW(); test_accel1(); test_accel2(); test_PrivateExtractIcons(); diff --git a/dlls/user32/tests/resource.rc b/dlls/user32/tests/resource.rc index 31ec68f..f01c6f5 100644 --- a/dlls/user32/tests/resource.rc +++ b/dlls/user32/tests/resource.rc @@ -41,6 +41,7 @@ STRINGTABLE { 0 "String resource" 1 "Another string resource" + 2 L"This is a wide string resource" 65534 "Test high id" }

1 0

Alexandre Julliard : server: Invalidate the newly exposed child region on the parent too if it doesn' t clip children.
by Alexandre Julliard 03 Mar '08

03 Mar '08

Module: wine Branch: master Commit: e42eaaaf3a873d29b14f8f3a3617df2d0d4210ca URL: http://source.winehq.org/git/wine.git/?a=commit;h=e42eaaaf3a873d29b14f8f3a3… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 3 17:52:18 2008 +0100 server: Invalidate the newly exposed child region on the parent too if it doesn't clip children. --- server/window.c | 33 ++++++++++++++++++++------------- 1 files changed, 20 insertions(+), 13 deletions(-) diff --git a/server/window.c b/server/window.c index 5b8e69b..2633cd2 100644 --- a/server/window.c +++ b/server/window.c @@ -1363,8 +1363,8 @@ static unsigned int get_window_update_flags( struct window *win, struct window * static struct region *expose_window( struct window *win, const rectangle_t *old_window_rect, struct region *old_vis_rgn ) { - struct window *parent = win; struct region *new_vis_rgn, *exposed_rgn; + int offset_x, offset_y; if (!(new_vis_rgn = get_visible_region( win, DCX_WINDOW ))) return NULL; @@ -1383,23 +1383,30 @@ static struct region *expose_window( struct window *win, const rectangle_t *old_ } } - /* make it relative to the old window pos for subtracting */ - offset_region( new_vis_rgn, win->window_rect.left - old_window_rect->left, - win->window_rect.top - old_window_rect->top ); - - if (subtract_region( new_vis_rgn, old_vis_rgn, new_vis_rgn ) && !is_region_empty( new_vis_rgn )) + offset_x = old_window_rect->left - win->client_rect.left; + offset_y = old_window_rect->top - win->client_rect.top; + if (win->parent) { - /* make it relative to new client rect again */ - int offset_x = old_window_rect->left - win->client_rect.left; - int offset_y = old_window_rect->top - win->client_rect.top; - if (win->parent && !is_desktop_window(win->parent)) + if (!is_desktop_window(win->parent)) { offset_x += win->client_rect.left; offset_y += win->client_rect.top; - parent = win->parent; } - offset_region( new_vis_rgn, offset_x, offset_y ); - redraw_window( parent, new_vis_rgn, 0, RDW_INVALIDATE | RDW_ERASE | RDW_ALLCHILDREN ); + + /* make it relative to the old window pos for subtracting */ + offset_region( new_vis_rgn, win->window_rect.left - old_window_rect->left, + win->window_rect.top - old_window_rect->top ); + + if ((win->parent->style & WS_CLIPCHILDREN) ? + subtract_region( new_vis_rgn, old_vis_rgn, new_vis_rgn ) : + xor_region( new_vis_rgn, old_vis_rgn, new_vis_rgn )) + { + if (!is_region_empty( new_vis_rgn )) + { + offset_region( new_vis_rgn, offset_x, offset_y ); + redraw_window( win->parent, new_vis_rgn, 0, RDW_INVALIDATE | RDW_ERASE | RDW_ALLCHILDREN ); + } + } } free_region( new_vis_rgn ); return exposed_rgn;

1 0

Alexandre Julliard : user32/tests: Skip tests instead of crashing if global hook cannot be set.
by Alexandre Julliard 03 Mar '08

03 Mar '08

Module: wine Branch: master Commit: 9ac7f205141bba9836d0f1f4afa4627078efb294 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9ac7f205141bba9836d0f1f4a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 3 17:51:42 2008 +0100 user32/tests: Skip tests instead of crashing if global hook cannot be set. --- dlls/user32/tests/msg.c | 7 ++++++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index 575d5d0..feb5d77 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -7239,7 +7239,12 @@ static void test_winevents(void) /****** start of global hook test *************/ hCBT_global_hook = SetWindowsHookExA(WH_CBT, cbt_global_hook_proc, GetModuleHandleA(0), 0); - assert(hCBT_global_hook); + if (!hCBT_global_hook) + { + ok(DestroyWindow(hwnd), "failed to destroy window\n"); + skip( "cannot set global hook\n" ); + return; + } hevent = CreateEventA(NULL, 0, 0, NULL); assert(hevent);

1 0

Dmitry Timoshkov : fonts: Add small cyrillic glyphs to tahoma.sfd.
by Alexandre Julliard 03 Mar '08

03 Mar '08

Module: wine Branch: master Commit: 57786d84a103fc107eee59c5c1177f05adcd5610 URL: http://source.winehq.org/git/wine.git/?a=commit;h=57786d84a103fc107eee59c5c… Author: Dmitry Timoshkov <dmitry(a)codeweavers.com> Date: Mon Mar 3 18:14:25 2008 +0800 fonts: Add small cyrillic glyphs to tahoma.sfd. --- fonts/tahoma.sfd | 1294 +++++++++++++++++++++++++++++++++++++++++++++++++++--- 1 files changed, 1239 insertions(+), 55 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=57786d84a103fc107eee5…

1 0

Paul Vriens : advapi32/tests: Skip not available functions.
by Alexandre Julliard 03 Mar '08

03 Mar '08

Module: wine Branch: master Commit: bfc290f6d0779ce83017155703ea7a530e8a7e79 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bfc290f6d0779ce8301715570… Author: Paul Vriens <paul.vriens.wine(a)gmail.com> Date: Mon Mar 3 10:46:06 2008 +0100 advapi32/tests: Skip not available functions. --- dlls/advapi32/tests/security.c | 13 +++++++++++-- 1 files changed, 11 insertions(+), 2 deletions(-) diff --git a/dlls/advapi32/tests/security.c b/dlls/advapi32/tests/security.c index 5086cb7..9ff02e9 100644 --- a/dlls/advapi32/tests/security.c +++ b/dlls/advapi32/tests/security.c @@ -910,8 +910,17 @@ static void test_AccessCheck(void) /* test INHERIT_ONLY_ACE */ ret = InitializeAcl(Acl, 256, ACL_REVISION); ok(ret, "InitializeAcl failed with error %d\n", GetLastError()); - ret = pAddAccessAllowedAceEx(Acl, ACL_REVISION, INHERIT_ONLY_ACE, KEY_READ, EveryoneSid); - ok(ret, "AddAccessAllowedAceEx failed with error %d\n", GetLastError()); + + /* NT doesn't have AddAccessAllowedAceEx. Skipping this call/test doesn't influence + * the next ones. + */ + if (pAddAccessAllowedAceEx) + { + ret = pAddAccessAllowedAceEx(Acl, ACL_REVISION, INHERIT_ONLY_ACE, KEY_READ, EveryoneSid); + ok(ret, "AddAccessAllowedAceEx failed with error %d\n", GetLastError()); + } + else + skip("AddAccessAllowedAceEx is not available\n"); ret = AccessCheck(SecurityDescriptor, Token, KEY_READ, &Mapping, PrivSet, &PrivSetLen, &Access, &AccessStatus);

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits March 2008