ChangeSet ID: 21211 CVSROOT: /opt/cvs-commit Module name: wine Changes by: julliard@winehq.org 2005/11/10 05:17:38
Modified files: dlls/user/tests: msg.c dlls/setupapi/tests: parser.c dlls/msxml3/tests: domdoc.c dlls/kernel/tests: thread.c dlls/d3d9 : device.c dlls/comctl32/tests: toolbar.c
Log message: Francois Gouget fgouget@free.fr Add trailing '\n's to ok() and TRACE() calls.
Patch: http://cvs.winehq.org/patch.py?id=21211
Old revision New revision Changes Path 1.101 1.102 +2 -2 wine/dlls/user/tests/msg.c 1.1 1.2 +3 -3 wine/dlls/setupapi/tests/parser.c 1.9 1.10 +3 -3 wine/dlls/msxml3/tests/domdoc.c 1.28 1.29 +2 -2 wine/dlls/kernel/tests/thread.c 1.31 1.32 +1 -1 wine/dlls/d3d9/device.c 1.1 1.2 +1 -1 wine/dlls/comctl32/tests/toolbar.c
Index: wine/dlls/user/tests/msg.c diff -u -p wine/dlls/user/tests/msg.c:1.101 wine/dlls/user/tests/msg.c:1.102 --- wine/dlls/user/tests/msg.c:1.101 10 Nov 2005 11:17:38 -0000 +++ wine/dlls/user/tests/msg.c 10 Nov 2005 11:17:38 -0000 @@ -3735,7 +3735,7 @@ static void test_paint_messages(void) */ trace("testing ValidateRect(0, NULL)\n"); SetRectEmpty( &rect ); - ok(ValidateRect(0, &rect), "ValidateRect(0, &rc) should not fail"); + ok(ValidateRect(0, &rect), "ValidateRect(0, &rc) should not fail\n"); check_update_rgn( hwnd, hrgn ); ok_sequence( WmInvalidateErase, "InvalidateErase", FALSE ); while (PeekMessage( &msg, 0, 0, 0, PM_REMOVE )) DispatchMessage( &msg ); @@ -3753,7 +3753,7 @@ static void test_paint_messages(void)
trace("testing ValidateRgn(0, NULL)\n"); SetLastError(0xdeadbeef); - ok(!ValidateRgn(0, NULL), "ValidateRgn(0, NULL) should fail"); + ok(!ValidateRgn(0, NULL), "ValidateRgn(0, NULL) should fail\n"); ok(GetLastError() == ERROR_INVALID_WINDOW_HANDLE, "wrong error code %ld\n", GetLastError()); check_update_rgn( hwnd, 0 ); while (PeekMessage( &msg, 0, 0, 0, PM_REMOVE )) DispatchMessage( &msg ); Index: wine/dlls/setupapi/tests/parser.c diff -u -p wine/dlls/setupapi/tests/parser.c:1.1 wine/dlls/setupapi/tests/parser.c:1.2 --- wine/dlls/setupapi/tests/parser.c:1.1 10 Nov 2005 11:17:38 -0000 +++ wine/dlls/setupapi/tests/parser.c 10 Nov 2005 11:17:38 -0000 @@ -218,13 +218,13 @@ static void test_section_names(void) trace( "hinf=%p ret=%ld err=%lx\n", hinf, ret, err ); if (ret != -1) { - ok( !section_names[i].error, "line %u: section name %s found", + ok( !section_names[i].error, "line %u: section name %s found\n", i, section_names[i].section ); - ok( !err, "line %u: bad error code %lx", i, err ); + ok( !err, "line %u: bad error code %lx\n", i, err ); } else { - ok( section_names[i].error, "line %u: section name %s not found", + ok( section_names[i].error, "line %u: section name %s not found\n", i, section_names[i].section ); ok( err == section_names[i].error, "line %u: bad error %lx/%lx\n", i, err, section_names[i].error ); Index: wine/dlls/msxml3/tests/domdoc.c diff -u -p wine/dlls/msxml3/tests/domdoc.c:1.9 wine/dlls/msxml3/tests/domdoc.c:1.10 --- wine/dlls/msxml3/tests/domdoc.c:1.9 10 Nov 2005 11:17:39 -0000 +++ wine/dlls/msxml3/tests/domdoc.c 10 Nov 2005 11:17:39 -0000 @@ -157,7 +157,7 @@ void test_domdoc( void )
/* check if nodename is correct */ r = IXMLDOMDocument_get_nodeName( doc, NULL ); - ok ( r == E_INVALIDARG, "get_nodeName (NULL) wrong code"); + ok ( r == E_INVALIDARG, "get_nodeName (NULL) wrong code\n");
/* content doesn't matter here */ str = SysAllocString( szNonExistentFile ); @@ -307,7 +307,7 @@ void test_domnode( void )
/* check if nodename is correct */ r = IXMLDOMElement_get_nodeName( element, NULL ); - ok ( r == E_INVALIDARG, "get_nodeName (NULL) wrong code"); + ok ( r == E_INVALIDARG, "get_nodeName (NULL) wrong code\n");
/* content doesn't matter here */ str = SysAllocString( szNonExistentFile ); @@ -358,7 +358,7 @@ void test_domnode( void ) ok ( node != NULL, "should be attribute\n");
r = IXMLDOMNode_get_nodeName( node, NULL ); - ok ( r == E_INVALIDARG, "get_nodeName (NULL) wrong code"); + ok ( r == E_INVALIDARG, "get_nodeName (NULL) wrong code\n");
/* content doesn't matter here */ str = SysAllocString( szNonExistentFile ); Index: wine/dlls/kernel/tests/thread.c diff -u -p wine/dlls/kernel/tests/thread.c:1.28 wine/dlls/kernel/tests/thread.c:1.29 --- wine/dlls/kernel/tests/thread.c:1.28 10 Nov 2005 11:17:39 -0000 +++ wine/dlls/kernel/tests/thread.c 10 Nov 2005 11:17:39 -0000 @@ -236,7 +236,7 @@ static VOID test_CreateThread_basic(void thread[0] = CreateThread(NULL,0,threadFunc2,NULL,0,NULL); GLE = GetLastError(); if (thread[0]) { /* NT */ - ok(GLE==0xFACEaBAD, "CreateThread set last error to %ld, expected 4207848365", GLE); + ok(GLE==0xFACEaBAD, "CreateThread set last error to %ld, expected 4207848365\n", GLE); ret = WaitForSingleObject(thread[0],100); ok(ret==WAIT_OBJECT_0, "threadFunc2 did not exit during 100 ms\n"); ret = GetExitCodeThread(thread[0],&exitCode); @@ -245,7 +245,7 @@ static VOID test_CreateThread_basic(void ok(CloseHandle(thread[0])!=0,"Error closing thread handle\n"); } else { /* 9x */ - ok(GLE==ERROR_INVALID_PARAMETER, "CreateThread set last error to %ld, expected 87", GLE); + ok(GLE==ERROR_INVALID_PARAMETER, "CreateThread set last error to %ld, expected 87\n", GLE); } }
Index: wine/dlls/d3d9/device.c diff -u -p wine/dlls/d3d9/device.c:1.31 wine/dlls/d3d9/device.c:1.32 --- wine/dlls/d3d9/device.c:1.31 10 Nov 2005 11:17:39 -0000 +++ wine/dlls/d3d9/device.c 10 Nov 2005 11:17:39 -0000 @@ -104,7 +104,7 @@ HRESULT WINAPI IDirect3DDevice9Impl_Ge FIXME("Call to IWineD3DDevice_GetDirect3D failed\n"); *ppD3D9 = NULL; } - TRACE("(%p) returning %p\b",This , *ppD3D9); + TRACE("(%p) returning %p\n", This, *ppD3D9); return hr; }
Index: wine/dlls/comctl32/tests/toolbar.c diff -u -p wine/dlls/comctl32/tests/toolbar.c:1.1 wine/dlls/comctl32/tests/toolbar.c:1.2 --- wine/dlls/comctl32/tests/toolbar.c:1.1 10 Nov 2005 11:17:39 -0000 +++ wine/dlls/comctl32/tests/toolbar.c 10 Nov 2005 11:17:39 -0000 @@ -59,7 +59,7 @@ static LRESULT CALLBACK MyWndProc(HWND h 0, NULL, (UINT)0, buttons, sizeof(buttons)/sizeof(buttons[0]), 0, 0, 20, 16, sizeof(TBBUTTON)); - ok(hToolbar != NULL, "Toolbar creation"); + ok(hToolbar != NULL, "Toolbar creation\n");
SendMessage(hToolbar, TB_ADDSTRINGA, 0, (LPARAM)"test\000");