wine-commits February 2011

wine-commits@winehq.org

2 participants
674 discussions

Austin English : credui/tests: Make sure to use return value (LLVM/Clang).
by Alexandre Julliard 07 Feb '11

07 Feb '11

Module: wine Branch: master Commit: c56ce79160687d85b61d8c0304271e6a174436c0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c56ce79160687d85b61d8c030… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 4 16:05:09 2011 -0800 credui/tests: Make sure to use return value (LLVM/Clang). --- dlls/credui/tests/credui.c | 14 ++++++++++++++ 1 files changed, 14 insertions(+), 0 deletions(-) diff --git a/dlls/credui/tests/credui.c b/dlls/credui/tests/credui.c index 969b096..18fc5bc 100644 --- a/dlls/credui/tests/credui.c +++ b/dlls/credui/tests/credui.c @@ -83,7 +83,10 @@ static void test_CredUIPromptForCredentials(void) &save, CREDUI_FLAGS_EXPECT_CONFIRMATION); ok(ret == ERROR_SUCCESS || ret == ERROR_CANCELLED, "CredUIPromptForCredentials failed with error %d\n", ret); if (ret == ERROR_SUCCESS) + { ret = CredUIConfirmCredentialsW(wszServerName, FALSE); + ok(ret == ERROR_SUCCESS, "CredUIConfirmCredentials failed with error %d\n", ret); + } credui_info.pszCaptionText = wszCaption1; ret = CredUIPromptForCredentialsW(&credui_info, wszServerName, NULL, @@ -93,7 +96,10 @@ static void test_CredUIPromptForCredentials(void) &save, CREDUI_FLAGS_EXPECT_CONFIRMATION); ok(ret == ERROR_SUCCESS || ret == ERROR_CANCELLED, "CredUIPromptForCredentials failed with error %d\n", ret); if (ret == ERROR_SUCCESS) + { ret = CredUIConfirmCredentialsW(wszServerName, FALSE); + ok(ret == ERROR_SUCCESS, "CredUIConfirmCredentials failed with error %d\n", ret); + } credui_info.pszCaptionText = wszCaption2; ret = CredUIPromptForCredentialsW(&credui_info, wszServerName, NULL, 0, @@ -102,7 +108,11 @@ static void test_CredUIPromptForCredentials(void) NULL, CREDUI_FLAGS_INCORRECT_PASSWORD|CREDUI_FLAGS_EXPECT_CONFIRMATION); ok(ret == ERROR_SUCCESS || ret == ERROR_CANCELLED, "CredUIPromptForCredentials failed with error %d\n", ret); if (ret == ERROR_SUCCESS) + { ret = CredUIConfirmCredentialsW(wszServerName, FALSE); + ok(ret == ERROR_SUCCESS, "CredUIConfirmCredentials failed with error %d\n", ret); + } + save = TRUE; credui_info.pszCaptionText = wszCaption3; @@ -122,7 +132,11 @@ static void test_CredUIPromptForCredentials(void) ok(ret == ERROR_SUCCESS || ret == ERROR_CANCELLED, "CredUIPromptForCredentials failed with error %d\n", ret); ok(!save, "save flag should have been untouched\n"); if (ret == ERROR_SUCCESS) + { ret = CredUIConfirmCredentialsW(wszServerName, FALSE); + ok(ret == ERROR_SUCCESS, "CredUIConfirmCredentials failed with error %d\n", ret); + } + } }

1 0

Austin English : dinput/tests: Make sure to use return value (LLVM/Clang).
by Alexandre Julliard 07 Feb '11

07 Feb '11

Module: wine Branch: master Commit: d6d749f4d755a3dae162a34dd951d6f1f8c1903a URL: http://source.winehq.org/git/wine.git/?a=commit;h=d6d749f4d755a3dae162a34dd… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 4 16:01:15 2011 -0800 dinput/tests: Make sure to use return value (LLVM/Clang). --- dlls/dinput/tests/joystick.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/dinput/tests/joystick.c b/dlls/dinput/tests/joystick.c index a60d6e2..194069d 100644 --- a/dlls/dinput/tests/joystick.c +++ b/dlls/dinput/tests/joystick.c @@ -489,6 +489,7 @@ static BOOL CALLBACK EnumJoysticks( hr = IDirectInputDevice_Acquire(pJoystick); ok(hr==DI_OK,"IDirectInputDevice_Acquire() failed: %08x\n", hr); hr = IDirectInputDevice2_GetProperty(pJoystick, DIPROP_AUTOCENTER, &diprop_word.diph); + ok(hr==DI_OK,"IDirectInputDevice2_GetProperty() failed: %08x\n", hr); /* Device gain (DIPROP_FFGAIN). * From MSDN:

1 0

Austin English : comcat/tests: Make sure to use return value (LLVM/Clang).
by Alexandre Julliard 07 Feb '11

07 Feb '11

Module: wine Branch: master Commit: 453405e2b4a78fcf81a7c9407fd32ce100b45fb1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=453405e2b4a78fcf81a7c9407… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 4 15:46:45 2011 -0800 comcat/tests: Make sure to use return value (LLVM/Clang). --- dlls/comcat/tests/comcat.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/comcat/tests/comcat.c b/dlls/comcat/tests/comcat.c index 029d2e3..cf0b7c1 100644 --- a/dlls/comcat/tests/comcat.c +++ b/dlls/comcat/tests/comcat.c @@ -104,6 +104,7 @@ static void do_enum(void) ok_ole_success(hr,"ICatInformation_EnumClassesOfCategories"); hr = IEnumGUID_Next(pIEnum,1,the_guid, &fetched); + ok (hr == S_FALSE,"Expected S_FALSE, got 0x%08x", hr); ok (fetched == 0,"Fetched wrong number of guids %u\n",fetched); IEnumGUID_Release(pIEnum); @@ -113,6 +114,7 @@ static void do_enum(void) ok_ole_success(hr,"ICatInformation_EnumClassesOfCategories"); hr = IEnumGUID_Next(pIEnum,1,the_guid, &fetched); + ok (hr == S_OK,"Expected S_OK, got 0x%08x", hr); ok (fetched == 1,"Fetched wrong number of guids %u\n",fetched); ok (IsEqualGUID(the_guid,&wanted_guid),"Guids do not match\n");

1 0

Austin English : advapi32/tests: Make sure to use return value (LLVM/Clang) .
by Alexandre Julliard 07 Feb '11

07 Feb '11

Module: wine Branch: master Commit: 425e7ca15d21201b646510cdff6a3b9dd75c89ac URL: http://source.winehq.org/git/wine.git/?a=commit;h=425e7ca15d21201b646510cdf… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 4 15:44:40 2011 -0800 advapi32/tests: Make sure to use return value (LLVM/Clang). --- dlls/advapi32/tests/registry.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/advapi32/tests/registry.c b/dlls/advapi32/tests/registry.c index 42ff232..5316e95 100644 --- a/dlls/advapi32/tests/registry.c +++ b/dlls/advapi32/tests/registry.c @@ -1199,6 +1199,7 @@ static void test_reg_close_key(void) * hkHandle remains changed after call to RegCloseKey */ ret = RegOpenKeyA(HKEY_CURRENT_USER, "Software\\Wine\\Test", &hkHandle); + ok(ret == ERROR_SUCCESS, "expected ERROR_SUCCESS, got %d\n", ret); ret = RegCloseKey(hkHandle); ok(ret == ERROR_SUCCESS, "expected ERROR_SUCCESS, got %d\n", ret);

1 0

Stefan Dösinger : d3d9: Use a proper window class for the query test.
by Alexandre Julliard 07 Feb '11

07 Feb '11

Module: wine Branch: master Commit: eadae86c858fb132191a317ebfaf51b3d7d86226 URL: http://source.winehq.org/git/wine.git/?a=commit;h=eadae86c858fb132191a317eb… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sun Jan 30 20:41:56 2011 +0100 d3d9: Use a proper window class for the query test. --- dlls/d3d9/tests/query.c | 18 ++++++++++++------ 1 files changed, 12 insertions(+), 6 deletions(-) diff --git a/dlls/d3d9/tests/query.c b/dlls/d3d9/tests/query.c index d015353..8bcfcf6 100644 --- a/dlls/d3d9/tests/query.c +++ b/dlls/d3d9/tests/query.c @@ -255,25 +255,31 @@ START_TEST(query) { IDirect3D9 *pD3d = NULL; HWND hwnd = NULL; + WNDCLASS wc = {0}; + wc.lpfnWndProc = DefWindowProc; + wc.lpszClassName = "d3d9_test_wc"; + RegisterClass(&wc); pD3d = pDirect3DCreate9( D3D_SDK_VERSION ); if(!pD3d) { skip("Failed to create Direct3D9 object, not running tests\n"); - return; + goto out; } - hwnd = CreateWindow( "static", "d3d9_test", WS_OVERLAPPEDWINDOW, 100, 100, 160, 160, NULL, NULL, NULL, NULL ); + hwnd = CreateWindow( wc.lpszClassName, "d3d9_test", + WS_SYSMENU | WS_POPUP, 100, 100, 160, 160, NULL, NULL, NULL, NULL ); if(!hwnd) { skip("Failed to create window\n"); - IDirect3D9_Release(pD3d); - return; + goto out; } test_query_support(pD3d, hwnd); test_occlusion_query_states(pD3d, hwnd); - DestroyWindow(hwnd); - IDirect3D9_Release(pD3d); + out: + if(pD3d) IDirect3D9_Release(pD3d); + if(hwnd) DestroyWindow(hwnd); + UnregisterClassA(wc.lpszClassName, GetModuleHandleA(NULL)); } }

1 0

Joris Huizer : ws2_32/tests: Don' t test function directly when reporting GetLastError().
by Alexandre Julliard 07 Feb '11

07 Feb '11

Module: wine Branch: master Commit: 36c76573de03d5890ae746f5cbddbe9fb5e42092 URL: http://source.winehq.org/git/wine.git/?a=commit;h=36c76573de03d5890ae746f5c… Author: Joris Huizer <joris_huizer(a)yahoo.com> Date: Fri Feb 4 17:20:25 2011 +0100 ws2_32/tests: Don't test function directly when reporting GetLastError(). --- dlls/ws2_32/tests/sock.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/ws2_32/tests/sock.c b/dlls/ws2_32/tests/sock.c index 9d1de46..2a8c1d2 100644 --- a/dlls/ws2_32/tests/sock.c +++ b/dlls/ws2_32/tests/sock.c @@ -3773,7 +3773,8 @@ static void test_WSARecv(void) ling.l_onoff = 1; ling.l_linger = 0; - ok(!setsockopt (src, SOL_SOCKET, SO_LINGER, (char *) &ling, sizeof(ling)), "Failed to set linger %d\n", GetLastError()); + iret = setsockopt (src, SOL_SOCKET, SO_LINGER, (char *) &ling, sizeof(ling)); + ok(!iret, "Failed to set linger %d\n", GetLastError()); iret = WSARecv(dest, &bufs, 1, &bytesReturned, &flags, &ov, NULL); ok(iret == SOCKET_ERROR && GetLastError() == ERROR_IO_PENDING, "WSARecv failed - %d error %d\n", iret, GetLastError());

1 0

Joris Huizer : user32/tests: Don' t test function directly when reporting GetLastError().
by Alexandre Julliard 07 Feb '11

07 Feb '11

Module: wine Branch: master Commit: 26f753aec8c81e09938cc39383322aae3fdc4e72 URL: http://source.winehq.org/git/wine.git/?a=commit;h=26f753aec8c81e09938cc3938… Author: Joris Huizer <joris_huizer(a)yahoo.com> Date: Fri Feb 4 17:10:17 2011 +0100 user32/tests: Don't test function directly when reporting GetLastError(). --- dlls/user32/tests/dde.c | 7 +++++-- dlls/user32/tests/menu.c | 7 +++++-- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/dlls/user32/tests/dde.c b/dlls/user32/tests/dde.c index ea010e6..f6a0844 100644 --- a/dlls/user32/tests/dde.c +++ b/dlls/user32/tests/dde.c @@ -1571,6 +1571,7 @@ static void test_dde_aw_transaction( BOOL client_unicode, BOOL server_unicode ) CONVINFO info; HDDEDATA hdata; BOOL conv_unicode = client_unicode; + BOOL got; static char test_cmd[] = "test dde command"; if (!(hwnd_server = create_dde_server( server_unicode ))) return; @@ -1709,7 +1710,8 @@ todo_wine { ok(err == DMLERR_NOTPROCESSED, "DdeClientTransaction returned error %x\n", err); } - ok(DdeDisconnect(hconv), "DdeDisconnect error %x\n", DdeGetLastError(dde_inst)); + got = DdeDisconnect(hconv); + ok(got, "DdeDisconnect error %x\n", DdeGetLastError(dde_inst)); info.cb = sizeof(info); ret = DdeQueryConvInfo(hconv, QID_SYNC, &info); @@ -1719,7 +1721,8 @@ todo_wine { ok(err == DMLERR_INVALIDPARAMETER, "wrong dde error %x\n", err); } - ok(DdeFreeStringHandle(dde_inst, hsz_server), "DdeFreeStringHandle error %x\n", DdeGetLastError(dde_inst)); + got = DdeFreeStringHandle(dde_inst, hsz_server); + ok(got, "DdeFreeStringHandle error %x\n", DdeGetLastError(dde_inst)); /* This call hangs on win2k SP4 and XP SP1. DdeUninitialize(dde_inst);*/ diff --git a/dlls/user32/tests/menu.c b/dlls/user32/tests/menu.c index aa7ebde..6a831a8 100644 --- a/dlls/user32/tests/menu.c +++ b/dlls/user32/tests/menu.c @@ -660,6 +660,7 @@ static void test_menu_bmp_and_string(void) MENUINFO mi= {sizeof(MENUINFO)}; MENUITEMINFOA mii= {sizeof(MENUITEMINFOA)}; int count, szidx, txtidx, bmpidx, hassub, mnuopt, ispop; + BOOL got; if( !pGetMenuInfo) { @@ -681,12 +682,14 @@ static void test_menu_bmp_and_string(void) ok( hsysmenu != NULL, "GetSystemMenu failed with error %d\n", GetLastError()); mi.fMask = MIM_STYLE; mi.dwStyle = 0; - ok( pGetMenuInfo( hsysmenu, &mi), "GetMenuInfo failed gle=%d\n", GetLastError()); + got = pGetMenuInfo( hsysmenu, &mi); + ok( got, "GetMenuInfo failed gle=%d\n", GetLastError()); ok( MNS_CHECKORBMP == mi.dwStyle, "System Menu Style is %08x, without the bit %08x\n", mi.dwStyle, MNS_CHECKORBMP); mii.fMask = MIIM_BITMAP; mii.hbmpItem = NULL; - ok( GetMenuItemInfoA( hsysmenu, SC_CLOSE, FALSE, &mii), "GetMenuItemInfoA failed gle=%d\n", GetLastError()); + got = GetMenuItemInfoA( hsysmenu, SC_CLOSE, FALSE, &mii); + ok( got, "GetMenuItemInfoA failed gle=%d\n", GetLastError()); ok( HBMMENU_POPUP_CLOSE == mii.hbmpItem, "Item info did not get the right hbitmap: got %p expected %p\n", mii.hbmpItem, HBMMENU_POPUP_CLOSE);

1 0

Joris Huizer : kernel32/tests: Don' t test function directly when reporting GetLastError().
by Alexandre Julliard 07 Feb '11

07 Feb '11

Module: wine Branch: master Commit: fb296d120158e34896405bccf72ce2db659e0bed URL: http://source.winehq.org/git/wine.git/?a=commit;h=fb296d120158e34896405bccf… Author: Joris Huizer <joris_huizer(a)yahoo.com> Date: Fri Feb 4 16:49:14 2011 +0100 kernel32/tests: Don't test function directly when reporting GetLastError(). --- dlls/kernel32/tests/thread.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/kernel32/tests/thread.c b/dlls/kernel32/tests/thread.c index 6f27db1..fab01dc 100644 --- a/dlls/kernel32/tests/thread.c +++ b/dlls/kernel32/tests/thread.c @@ -944,7 +944,8 @@ static void test_SetThreadContext(void) ctx.Esp -= 2 * sizeof(int *); ctx.Eip = (DWORD)set_test_val; SetLastError(0xdeadbeef); - ok( SetThreadContext( thread, &ctx ), "SetThreadContext failed : (%d)\n", GetLastError() ); + ret = SetThreadContext( thread, &ctx ); + ok( ret, "SetThreadContext failed : (%d)\n", GetLastError() ); } SetLastError(0xdeadbeef);

1 0

Joris Huizer : imagehlp/tests: Don' t test function directly when reporting GetLastError().
by Alexandre Julliard 07 Feb '11

07 Feb '11

Module: wine Branch: master Commit: 922d1d88e85cd64960ae44998c280ce738c13b28 URL: http://source.winehq.org/git/wine.git/?a=commit;h=922d1d88e85cd64960ae44998… Author: Joris Huizer <joris_huizer(a)yahoo.com> Date: Fri Feb 4 16:36:49 2011 +0100 imagehlp/tests: Don't test function directly when reporting GetLastError(). --- dlls/imagehlp/tests/integrity.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/imagehlp/tests/integrity.c b/dlls/imagehlp/tests/integrity.c index 10af9fa..0785c2a 100644 --- a/dlls/imagehlp/tests/integrity.c +++ b/dlls/imagehlp/tests/integrity.c @@ -202,7 +202,8 @@ static void test_get_certificate(char *cert_data, int index) return; } - ok(ret = pImageGetCertificateData(hFile, index, cert, &cert_len), "Unable to retrieve certificate; err=%x\n", GetLastError()); + ret = pImageGetCertificateData(hFile, index, cert, &cert_len); + ok(ret, "Unable to retrieve certificate; err=%x\n", GetLastError()); ok(memcmp(cert->bCertificate, cert_data, cert_len - sizeof(WIN_CERTIFICATE)) == 0, "Certificate retrieved did not match original\n"); HeapFree(GetProcessHeap(), 0, cert);

1 0

André Hentschel : ntdll: Only use ST_FIX_ALIGN on solaris.
by Alexandre Julliard 07 Feb '11

07 Feb '11

Module: wine Branch: master Commit: e78bd6dfa11e177d1c766c1148797a59d5e6cebc URL: http://source.winehq.org/git/wine.git/?a=commit;h=e78bd6dfa11e177d1c766c114… Author: André Hentschel <nerv(a)dawncrow.de> Date: Fri Feb 4 21:44:46 2011 +0100 ntdll: Only use ST_FIX_ALIGN on solaris. --- dlls/ntdll/signal_sparc.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/ntdll/signal_sparc.c b/dlls/ntdll/signal_sparc.c index 318e5a9..29414d7 100644 --- a/dlls/ntdll/signal_sparc.c +++ b/dlls/ntdll/signal_sparc.c @@ -809,6 +809,7 @@ void signal_init_process(void) if (sigaction( SIGTRAP, &sig_act, NULL ) == -1) goto error; #endif +#ifdef __sun__ /* 'ta 6' tells the kernel to synthesize any unaligned accesses this process makes, instead of just signalling an error and terminating the process. wine-devel did not reach a conclusion on whether @@ -816,6 +817,7 @@ void signal_init_process(void) because it could obscure problems in user code */ __asm__("ta 6"); /* 6 == ST_FIX_ALIGN defined in sys/trap.h */ return; +#endif error: perror("sigaction");

1 0

← Newer
1
...
53
54
55
56
57
58
59
...
68
Older →

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 February 2011