January 2022 - Wine-commits

Fabian Maurer : kernel32/tests: Avoid "misleading indentation" warnings.
by Alexandre Julliard 27 Jan '22

27 Jan '22

Module: wine Branch: master Commit: d37028ae54d3f5c5211ea41d86ef5a7766826c1b URL: https://source.winehq.org/git/wine.git/?a=commit;h=d37028ae54d3f5c5211ea41d… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Thu Jan 27 20:16:01 2022 +0100 kernel32/tests: Avoid "misleading indentation" warnings. Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/tests/actctx.c | 8 ++++---- dlls/kernel32/tests/file.c | 6 +++--- dlls/kernel32/tests/loader.c | 6 +++--- dlls/kernel32/tests/locale.c | 2 +- dlls/kernel32/tests/mailslot.c | 2 +- dlls/kernel32/tests/sync.c | 6 +++--- dlls/kernel32/tests/thread.c | 2 +- 7 files changed, 16 insertions(+), 16 deletions(-) diff --git a/dlls/kernel32/tests/actctx.c b/dlls/kernel32/tests/actctx.c index e103dfdcf2b..c26eb08f306 100644 --- a/dlls/kernel32/tests/actctx.c +++ b/dlls/kernel32/tests/actctx.c @@ -2668,7 +2668,7 @@ todo_wine { SetLastError(0xdeadbeef); handle = CreateActCtxA(&actctx); ok(handle == INVALID_HANDLE_VALUE, "got handle %p\n", handle); -todo_wine + todo_wine ok(GetLastError() == ERROR_SXS_CANT_GEN_ACTCTX || broken(GetLastError() == ERROR_NOT_ENOUGH_MEMORY) /* XP, win2k3 */, "got error %d\n", GetLastError()); @@ -2770,7 +2770,7 @@ static void test_ZombifyActCtx(void) SetLastError(0xdeadbeef); ret = ZombifyActCtx(NULL); -todo_wine + todo_wine ok(!ret && GetLastError() == ERROR_INVALID_PARAMETER, "got %d, error %d\n", ret, GetLastError()); handle = create_manifest("test.manifest", testdep_manifest3, __LINE__); @@ -2800,7 +2800,7 @@ todo_wine ok(basicinfo.dwFlags == 0, "got %x\n", basicinfo.dwFlags); ret = ZombifyActCtx(handle); -todo_wine + todo_wine ok(ret, "got %d\n", ret); memset(&basicinfo, 0xff, sizeof(basicinfo)); @@ -2822,7 +2822,7 @@ todo_wine /* one more time */ ret = ZombifyActCtx(handle); -todo_wine + todo_wine ok(ret, "got %d\n", ret); ret = DeactivateActCtx(0, cookie); diff --git a/dlls/kernel32/tests/file.c b/dlls/kernel32/tests/file.c index 60fe532adc2..f8e49491a7d 100644 --- a/dlls/kernel32/tests/file.c +++ b/dlls/kernel32/tests/file.c @@ -5313,12 +5313,12 @@ static void test_SetFileInformationByHandle(void) /* test FileDispositionInfo, additional details already covered by ntdll tests */ SetLastError(0xdeadbeef); ret = pSetFileInformationByHandle(file, FileDispositionInfo, &dispinfo, 0); -todo_wine + todo_wine ok(!ret && GetLastError() == ERROR_BAD_LENGTH, "got %d, error %d\n", ret, GetLastError()); SetLastError(0xdeadbeef); ret = pSetFileInformationByHandle(file, FileBasicInfo, &basicinfo, 0); -todo_wine + todo_wine ok(!ret && GetLastError() == ERROR_BAD_LENGTH, "got %d, error %d\n", ret, GetLastError()); memset(&basicinfo, 0, sizeof(basicinfo)); @@ -5409,7 +5409,7 @@ static void test_SetFileRenameInfo(void) fri->FileNameLength = wcslen(tempFileTo1) * sizeof(WCHAR); memcpy(fri->FileName, tempFileTo1, fri->FileNameLength + sizeof(WCHAR)); ret = pSetFileInformationByHandle(file, FileRenameInfo, fri, size); -todo_wine + todo_wine ok(!ret && GetLastError() == ERROR_ACCESS_DENIED, "FileRenameInfo unexpected result %d\n", GetLastError()); CloseHandle(file); diff --git a/dlls/kernel32/tests/loader.c b/dlls/kernel32/tests/loader.c index f990d632f73..6b5099ea612 100644 --- a/dlls/kernel32/tests/loader.c +++ b/dlls/kernel32/tests/loader.c @@ -2442,7 +2442,7 @@ static BOOL WINAPI dll_entry_point(HINSTANCE hinst, DWORD reason, LPVOID param) * doesn't call the DLL entry point on process detach either. */ HeapLock(GetProcessHeap()); -todo_wine + todo_wine ok(0, "dll_entry_point: process should already deadlock\n"); break; } @@ -3308,7 +3308,7 @@ static void test_ExitProcess(void) ret = CreateProcessA(argv[0], cmdline, NULL, NULL, FALSE, 0, NULL, NULL, &si, &pi); ok(ret, "CreateProcess(%s) error %d\n", cmdline, GetLastError()); ret = WaitForSingleObject(pi.hProcess, 5000); -todo_wine + todo_wine ok(ret == WAIT_TIMEOUT || broken(ret == WAIT_OBJECT_0) /* XP */, "child process should fail to terminate\n"); if (ret != WAIT_OBJECT_0) { @@ -3318,7 +3318,7 @@ todo_wine ret = WaitForSingleObject(pi.hProcess, 1000); ok(ret == WAIT_OBJECT_0, "child process failed to terminate\n"); GetExitCodeProcess(pi.hProcess, &ret); -todo_wine + todo_wine ok(ret == 201 || broken(ret == 1) /* XP */, "expected exit code 201, got %u\n", ret); if (*child_failures) { diff --git a/dlls/kernel32/tests/locale.c b/dlls/kernel32/tests/locale.c index 2d327b7130c..8f48b89334c 100644 --- a/dlls/kernel32/tests/locale.c +++ b/dlls/kernel32/tests/locale.c @@ -5519,7 +5519,7 @@ static void test_GetThreadPreferredUILanguages(void) ok(!ret, "Expected GetThreadPreferredUILanguages to fail\n"); ok(GetLastError() == ERROR_INSUFFICIENT_BUFFER, "Expected error ERROR_INSUFFICIENT_BUFFER, got %d\n", GetLastError()); -todo_wine + todo_wine ok(size == size_id || size == size_id - 1 /* before win10 1809 */, "expected %u, got %u\n", size_id, size); HeapFree(GetProcessHeap(), 0, buf); diff --git a/dlls/kernel32/tests/mailslot.c b/dlls/kernel32/tests/mailslot.c index 9da960fba81..7e488566dbb 100644 --- a/dlls/kernel32/tests/mailslot.c +++ b/dlls/kernel32/tests/mailslot.c @@ -88,7 +88,7 @@ static int mailslot_test(void) SetLastError(0xdeadbeef); ret = ReadFile(hSlot, buffer, 0, &count, NULL); ok(!ret, "ReadFile should fail\n"); -todo_wine + todo_wine ok(GetLastError() == ERROR_SEM_TIMEOUT, "wrong error %u\n", GetLastError()); ok(count == 0, "expected 0, got %u\n", count); diff --git a/dlls/kernel32/tests/sync.c b/dlls/kernel32/tests/sync.c index 827f4c53aa0..a6ee90b81f3 100644 --- a/dlls/kernel32/tests/sync.c +++ b/dlls/kernel32/tests/sync.c @@ -202,9 +202,9 @@ static void test_mutex(void) SetLastError(0xdeadbeef); hOpened = OpenMutexA(0, FALSE, "WineTestMutex"); -todo_wine + todo_wine ok(hOpened == NULL, "OpenMutex succeeded\n"); -todo_wine + todo_wine ok(GetLastError() == ERROR_ACCESS_DENIED, "wrong error %u\n", GetLastError()); SetLastError(0xdeadbeef); @@ -249,7 +249,7 @@ todo_wine } } -todo_wine + todo_wine ok( failed == 0x0de0fffe, "open succeeded when it shouldn't: %x\n", failed); SetLastError(0xdeadbeef); diff --git a/dlls/kernel32/tests/thread.c b/dlls/kernel32/tests/thread.c index e9fd90b1889..6ccf7af85f8 100644 --- a/dlls/kernel32/tests/thread.c +++ b/dlls/kernel32/tests/thread.c @@ -2338,7 +2338,7 @@ static void test_thread_info(void) break; case ThreadIsIoPending: -todo_wine + todo_wine ok(status == STATUS_ACCESS_DENIED, "for info %u expected STATUS_ACCESS_DENIED, got %08x (ret_len %u)\n", i, status, ret_len); break;

1 0

Fabian Maurer : msscript/tests: Avoid "misleading indentation" warnings.
by Alexandre Julliard 27 Jan '22

27 Jan '22

Module: wine Branch: master Commit: b8ac2441ed6e1228ea66335d9e3e73ecf69b64f2 URL: https://source.winehq.org/git/wine.git/?a=commit;h=b8ac2441ed6e1228ea66335d… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Thu Jan 27 20:16:09 2022 +0100 msscript/tests: Avoid "misleading indentation" warnings. Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msscript.ocx/tests/msscript.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/dlls/msscript.ocx/tests/msscript.c b/dlls/msscript.ocx/tests/msscript.c index 756c21d887b..897bcec3262 100644 --- a/dlls/msscript.ocx/tests/msscript.c +++ b/dlls/msscript.ocx/tests/msscript.c @@ -1453,10 +1453,10 @@ static void test_Language(void) CHECK_CALLED(QI_IActiveScriptParse); CHECK_CALLED(InitNew); hr = IScriptControl_get_Language(sc, &str); - todo_wine + todo_wine ok(hr == S_OK, "got 0x%08x\n", hr); - if (hr == S_OK) - ok(!lstrcmpW(L"testscript", str), "%s\n", wine_dbgstr_w(str)); + if (hr == S_OK) + ok(!lstrcmpW(L"testscript", str), "%s\n", wine_dbgstr_w(str)); SysFreeString(str); IActiveScriptSite_Release(site); @@ -2437,7 +2437,7 @@ static void _check_error(IScriptControl *sc, LONG exp_num, BOOL todo, int line) error_num = 0xdeadbeef; hr = IScriptError_get_Number(script_err, &error_num); ok_(__FILE__,line)(hr == S_OK, "IScriptError_get_Number failed: 0x%08x.\n", hr); -todo_wine_if(todo == TRUE) + todo_wine_if(todo == TRUE) ok_(__FILE__,line)(error_num == exp_num, "got wrong error number: %d, expected %d.\n", error_num, exp_num); IScriptError_Release(script_err);

1 0

Fabian Maurer : msvfw32/tests: Avoid "misleading indentation" warnings.
by Alexandre Julliard 27 Jan '22

27 Jan '22

Module: wine Branch: master Commit: d8b37d6628a6a808007ba547d51ff6deedb2b775 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d8b37d6628a6a808007ba547… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Thu Jan 27 20:16:11 2022 +0100 msvfw32/tests: Avoid "misleading indentation" warnings. Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msvfw32/tests/msvfw.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/msvfw32/tests/msvfw.c b/dlls/msvfw32/tests/msvfw.c index def78f02e9d..07ff7e3694c 100644 --- a/dlls/msvfw32/tests/msvfw.c +++ b/dlls/msvfw32/tests/msvfw.c @@ -477,7 +477,7 @@ static void test_ICInfo(void) ok(!lstrcmpW(info.szDriver, L"bogus"), "Got unexpected driver %s.\n", wine_dbgstr_w(info.szDriver)); /* Drivers installed after msvfw32 is loaded are not enumerated. */ -todo_wine + todo_wine ok(!ICInfo(test_type, 0, &info), "Expected failure.\n"); ret = ICRemove(test_type, test_handler, 0); @@ -505,7 +505,7 @@ todo_wine ok(!lstrcmpW(info.szDriver, L"bogus"), "Got unexpected driver %s.\n", wine_dbgstr_w(info.szDriver)); /* Drivers installed after msvfw32 is loaded are not enumerated. */ -todo_wine + todo_wine ok(!ICInfo(test_type, 0, &info), "Expected failure.\n"); ret = WritePrivateProfileStringA("drivers32", "wine.test", NULL, "system.ini");

1 0

Fabian Maurer : ntdll/tests: Avoid "misleading indentation" warnings.
by Alexandre Julliard 27 Jan '22

27 Jan '22

Module: wine Branch: master Commit: 0ea07db6ae34ae0431ab7975fc73926c19251de1 URL: https://source.winehq.org/git/wine.git/?a=commit;h=0ea07db6ae34ae0431ab7975… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Thu Jan 27 20:16:14 2022 +0100 ntdll/tests: Avoid "misleading indentation" warnings. Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/tests/file.c | 38 +++++++++++++++++++------------------- dlls/ntdll/tests/info.c | 2 +- dlls/ntdll/tests/reg.c | 6 +++--- 3 files changed, 23 insertions(+), 23 deletions(-) diff --git a/dlls/ntdll/tests/file.c b/dlls/ntdll/tests/file.c index 19ae5f2ac21..760e2ff167e 100644 --- a/dlls/ntdll/tests/file.c +++ b/dlls/ntdll/tests/file.c @@ -1113,7 +1113,7 @@ static void test_file_io_completion(void) U(iosb).Status = 0xdeadbeef; res = pNtSetInformationFile( server, &iosb, &fci, sizeof(fci), FileCompletionInformation ); ok( res == STATUS_INVALID_PARAMETER, "NtSetInformationFile failed: %#x\n", res ); -todo_wine + todo_wine ok( U(iosb).Status == 0xdeadbeef, "wrong status %#x\n", U(iosb).Status ); CloseHandle( client ); CloseHandle( server ); @@ -3020,20 +3020,20 @@ static void test_file_disposition_information(void) res = pNtSetInformationFile( handle, &io, &fdi, sizeof fdi, FileDispositionInformation ); ok( res == STATUS_SUCCESS, "unexpected FileDispositionInformation result (expected STATUS_SUCCESS, got %x)\n", res ); res = nt_get_file_attrs( buffer, &fdi2 ); -todo_wine + todo_wine ok( res == STATUS_DELETE_PENDING, "got %#x\n", res ); /* can't open the deleted file */ handle3 = CreateFileA(buffer, DELETE, FILE_SHARE_DELETE | FILE_SHARE_WRITE, NULL, OPEN_EXISTING, 0, 0); -todo_wine + todo_wine ok( handle3 == INVALID_HANDLE_VALUE, "CreateFile should fail\n" ); -if (handle3 != INVALID_HANDLE_VALUE) - CloseHandle( handle3 ); -todo_wine + if (handle3 != INVALID_HANDLE_VALUE) + CloseHandle( handle3 ); + todo_wine ok(GetLastError() == ERROR_ACCESS_DENIED, "got %u\n", GetLastError()); /* can't open the deleted file (wrong sharing mode) */ handle3 = CreateFileA(buffer, DELETE, 0, NULL, OPEN_EXISTING, 0, 0); ok( handle3 == INVALID_HANDLE_VALUE, "CreateFile should fail\n" ); -todo_wine + todo_wine ok(GetLastError() == ERROR_ACCESS_DENIED, "got %u\n", GetLastError()); CloseHandle( handle ); fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; @@ -3153,12 +3153,12 @@ todo_wine fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; ok( !fileDeleted, "File shouldn't have been deleted\n" ); res = nt_get_file_attrs( buffer, &fdi2 ); -todo_wine + todo_wine ok( res == STATUS_DELETE_PENDING, "got %#x\n", res ); /* can't open the deleted file */ handle2 = CreateFileA(buffer, DELETE, FILE_SHARE_DELETE, NULL, OPEN_EXISTING, 0, 0); ok( handle2 == INVALID_HANDLE_VALUE, "CreateFile should fail\n" ); -todo_wine + todo_wine ok(GetLastError() == ERROR_ACCESS_DENIED, "got %u\n", GetLastError()); CloseHandle( handle ); fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; @@ -3202,13 +3202,13 @@ todo_wine fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; ok( !fileDeleted, "Directory shouldn't have been deleted\n" ); res = nt_get_file_attrs( buffer, &fdi2 ); -todo_wine + todo_wine ok( res == STATUS_DELETE_PENDING, "got %#x\n", res ); /* can't open the deleted directory */ handle2 = CreateFileA(buffer, DELETE, FILE_SHARE_DELETE, NULL, OPEN_EXISTING, FILE_FLAG_BACKUP_SEMANTICS, 0); -todo_wine + todo_wine ok( handle2 == INVALID_HANDLE_VALUE, "CreateFile should fail\n" ); -todo_wine + todo_wine ok(GetLastError() == ERROR_ACCESS_DENIED, "got %u\n", GetLastError()); if (handle2 != INVALID_HANDLE_VALUE) CloseHandle( handle2 ); CloseHandle( handle ); @@ -3227,20 +3227,20 @@ todo_wine res = pNtSetInformationFile( handle2, &io, &fdi, sizeof fdi, FileDispositionInformation ); ok( res == STATUS_SUCCESS, "unexpected FileDispositionInformation result (expected STATUS_SUCCESS, got %x)\n", res ); res = nt_get_file_attrs( buffer, &fdi2 ); -todo_wine + todo_wine ok( res == STATUS_DELETE_PENDING, "got %#x\n", res ); /* can't open the deleted directory */ handle3 = CreateFileA(buffer, DELETE, FILE_SHARE_DELETE, NULL, OPEN_EXISTING, FILE_FLAG_BACKUP_SEMANTICS, 0); -todo_wine + todo_wine ok( handle3 == INVALID_HANDLE_VALUE, "CreateFile should fail\n" ); -if (handle3 != INVALID_HANDLE_VALUE) - CloseHandle( handle3 ); -todo_wine + if (handle3 != INVALID_HANDLE_VALUE) + CloseHandle( handle3 ); + todo_wine ok(GetLastError() == ERROR_ACCESS_DENIED, "got %u\n", GetLastError()); /* can't open the deleted directory (wrong sharing mode) */ handle3 = CreateFileA(buffer, DELETE, 0, NULL, OPEN_EXISTING, FILE_FLAG_BACKUP_SEMANTICS, 0); ok( handle3 == INVALID_HANDLE_VALUE, "CreateFile should fail\n" ); -todo_wine + todo_wine ok(GetLastError() == ERROR_ACCESS_DENIED, "got %u\n", GetLastError()); CloseHandle( handle2 ); fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; @@ -4231,7 +4231,7 @@ static void test_NtCreateFile(void) /* FIXME: leave only 'else' case below once Wine is fixed */ if (ret != td[i].attrib_out) { - todo_wine + todo_wine ok(ret == td[i].attrib_out, "%d: expected %#x got %#x\n", i, td[i].attrib_out, ret); SetFileAttributesW(path, td[i].attrib_out); } diff --git a/dlls/ntdll/tests/info.c b/dlls/ntdll/tests/info.c index 5bcccd8a322..e9d42127c3e 100644 --- a/dlls/ntdll/tests/info.c +++ b/dlls/ntdll/tests/info.c @@ -2251,7 +2251,7 @@ static void test_query_process_image_file_name(void) buffer = heap_alloc(ReturnLength); status = pNtQueryInformationProcess( GetCurrentProcess(), ProcessImageFileName, buffer, ReturnLength, &ReturnLength); ok( status == STATUS_SUCCESS, "Expected STATUS_SUCCESS, got %08x\n", status); -todo_wine + todo_wine ok(!memcmp(buffer->Buffer, deviceW, sizeof(deviceW)), "Expected image name to begin with \\Device\\, got %s\n", wine_dbgstr_wn(buffer->Buffer, buffer->Length / sizeof(WCHAR))); diff --git a/dlls/ntdll/tests/reg.c b/dlls/ntdll/tests/reg.c index 91b752ac069..bd8d531edf0 100644 --- a/dlls/ntdll/tests/reg.c +++ b/dlls/ntdll/tests/reg.c @@ -374,9 +374,9 @@ static void test_NtOpenKey(void) attr.Length = sizeof(attr); key = (HANDLE)0xdeadbeef; status = pNtOpenKey(&key, 0, &attr); -todo_wine + todo_wine ok(status == STATUS_ACCESS_DENIED, "Expected STATUS_ACCESS_DENIED, got: 0x%08x\n", status); -todo_wine + todo_wine ok(!key, "key = %p\n", key); if (status == STATUS_SUCCESS) NtClose(key); @@ -386,7 +386,7 @@ todo_wine key = (HANDLE)0xdeadbeef; status = pNtOpenKey(&key, KEY_READ, &attr); todo_wine ok(status == STATUS_OBJECT_PATH_SYNTAX_BAD, "NtOpenKey Failed: 0x%08x\n", status); -todo_wine + todo_wine ok(!key, "key = %p\n", key); pRtlFreeUnicodeString( &str );

1 0

Fabian Maurer : ntoskrnl/tests: Avoid "misleading indentation" warnings.
by Alexandre Julliard 27 Jan '22

27 Jan '22

Module: wine Branch: master Commit: 56412c1145f3a30504b5c56be4a63655b063a1b2 URL: https://source.winehq.org/git/wine.git/?a=commit;h=56412c1145f3a30504b5c56b… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Thu Jan 27 20:16:15 2022 +0100 ntoskrnl/tests: Avoid "misleading indentation" warnings. Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntoskrnl.exe/tests/driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/ntoskrnl.exe/tests/driver.c b/dlls/ntoskrnl.exe/tests/driver.c index 8cb97507ab5..0364fecab1d 100644 --- a/dlls/ntoskrnl.exe/tests/driver.c +++ b/dlls/ntoskrnl.exe/tests/driver.c @@ -280,7 +280,7 @@ static void test_mdl_map(void) MmProbeAndLockPages(mdl, KernelMode, IoReadAccess); addr = MmMapLockedPagesSpecifyCache(mdl, KernelMode, MmCached, NULL, FALSE, NormalPagePriority); -todo_wine + todo_wine ok(addr != NULL, "MmMapLockedPagesSpecifyCache failed\n"); MmUnmapLockedPages(addr, mdl);

1 0

Fabian Maurer : ole32/tests: Avoid "misleading indentation" warnings.
by Alexandre Julliard 27 Jan '22

27 Jan '22

Module: wine Branch: master Commit: 812944ec175245386e3af24af86ca8ad7c5c0cc6 URL: https://source.winehq.org/git/wine.git/?a=commit;h=812944ec175245386e3af24a… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Thu Jan 27 20:18:03 2022 +0100 ole32/tests: Avoid "misleading indentation" warnings. Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ole32/tests/compobj.c | 4 +- dlls/ole32/tests/defaulthandler.c | 8 ++-- dlls/ole32/tests/marshal.c | 2 +- dlls/ole32/tests/moniker.c | 88 +++++++++++++++++++-------------------- dlls/ole32/tests/ole2.c | 12 +++--- dlls/ole32/tests/ole_server.c | 4 +- dlls/ole32/tests/stg_prop.c | 10 ++--- dlls/ole32/tests/usrmarshal.c | 4 +- 8 files changed, 66 insertions(+), 66 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=812944ec175245386e3a…

1 0

Fabian Maurer : oleaut32/tests: Avoid "misleading indentation" warnings.
by Alexandre Julliard 27 Jan '22

27 Jan '22

Module: wine Branch: master Commit: 4a611ef59bfa05022a019588e8ba24d45e26f6d5 URL: https://source.winehq.org/git/wine.git/?a=commit;h=4a611ef59bfa05022a019588… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Thu Jan 27 20:18:04 2022 +0100 oleaut32/tests: Avoid "misleading indentation" warnings. Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/oleaut32/tests/olefont.c | 4 ++-- dlls/oleaut32/tests/olepicture.c | 22 +++++++++++----------- dlls/oleaut32/tests/safearray.c | 4 ++-- dlls/oleaut32/tests/tmarshal.c | 26 +++++++++++++------------- dlls/oleaut32/tests/typelib.c | 4 ++-- dlls/oleaut32/tests/usrmarshal.c | 8 ++++---- dlls/oleaut32/vartype.c | 6 +++--- 7 files changed, 37 insertions(+), 37 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=4a611ef59bfa05022a01…

1 0

Fabian Maurer : oledb32/tests: Avoid "misleading indentation" warnings.
by Alexandre Julliard 27 Jan '22

27 Jan '22

Module: wine Branch: master Commit: ad2dc608c0b427a9a7d007a2c519e771516b70f9 URL: https://source.winehq.org/git/wine.git/?a=commit;h=ad2dc608c0b427a9a7d007a2… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Thu Jan 27 20:18:05 2022 +0100 oledb32/tests: Avoid "misleading indentation" warnings. Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/oledb32/tests/convert.c | 34 +++++++++++++++++----------------- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/dlls/oledb32/tests/convert.c b/dlls/oledb32/tests/convert.c index 8ec64d40ae5..0d888e834e3 100644 --- a/dlls/oledb32/tests/convert.c +++ b/dlls/oledb32/tests/convert.c @@ -358,7 +358,7 @@ static void test_converttoi1(void) dst_len = dst = 0x12; *(int *)src = 0x4321cafe; hr = IDataConvert_DataConvert(convert, DBTYPE_I4, DBTYPE_I1, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); -todo_wine + todo_wine ok(hr == DB_E_DATAOVERFLOW, "got %08x\n", hr); ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); ok(dst_len == sizeof(dst), "got %ld\n", dst_len); @@ -492,7 +492,7 @@ todo_wine hr = IDataConvert_DataConvert(convert, DBTYPE_UI1, DBTYPE_I1, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); ok(hr == DB_E_ERRORSOCCURRED, "got %08x\n", hr); ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); -todo_wine + todo_wine ok(broken(dst_len == sizeof(dst)) || dst_len == 0x12 /* W2K+ */, "got %ld\n", dst_len); ok(dst == 0x12, "got %08x\n", dst); @@ -507,7 +507,7 @@ todo_wine dst_len = dst = 0x12; *(DWORD*)src = 0xabcd1234; hr = IDataConvert_DataConvert(convert, DBTYPE_UI4, DBTYPE_I1, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); -todo_wine + todo_wine ok(hr == DB_E_DATAOVERFLOW, "got %08x\n", hr); ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); ok(dst_len == sizeof(dst), "got %ld\n", dst_len); @@ -516,7 +516,7 @@ todo_wine dst_len = dst = 0x12; *(DWORD*)src = 0x12abcd; hr = IDataConvert_DataConvert(convert, DBTYPE_UI4, DBTYPE_I1, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); -todo_wine + todo_wine ok(hr == DB_E_DATAOVERFLOW, "got %08x\n", hr); ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); ok(dst_len == sizeof(dst), "got %ld\n", dst_len); @@ -602,7 +602,7 @@ todo_wine dst_len = dst = 0x12; *(DWORD*)src = 0xabcd1234; hr = IDataConvert_DataConvert(convert, DBTYPE_UI4, DBTYPE_UI1, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); -todo_wine + todo_wine ok(hr == DB_E_DATAOVERFLOW, "got %08x\n", hr); ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); ok(dst_len == sizeof(dst), "got %ld\n", dst_len); @@ -611,7 +611,7 @@ todo_wine dst_len = dst = 0x12; *(DWORD*)src = 0x12abcd; hr = IDataConvert_DataConvert(convert, DBTYPE_UI4, DBTYPE_UI1, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); -todo_wine + todo_wine ok(hr == DB_E_DATAOVERFLOW, "got %08x\n", hr); ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); ok(dst_len == sizeof(dst), "got %ld\n", dst_len); @@ -669,7 +669,7 @@ static void test_converttoi2(void) dst_len = dst = 0x1234; *(int *)src = 0x4321cafe; hr = IDataConvert_DataConvert(convert, DBTYPE_I4, DBTYPE_I2, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); -todo_wine + todo_wine ok(hr == DB_E_DATAOVERFLOW, "got %08x\n", hr); ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); ok(dst_len == sizeof(dst), "got %ld\n", dst_len); @@ -811,14 +811,14 @@ todo_wine hr = IDataConvert_DataConvert(convert, DBTYPE_UI2, DBTYPE_I2, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); ok(hr == DB_E_ERRORSOCCURRED, "got %08x\n", hr); ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); -todo_wine + todo_wine ok(broken(dst_len == sizeof(dst)) || dst_len == 0x1234 /* W2K+ */, "got %ld\n", dst_len); ok(dst == 0x1234, "got %08x\n", dst); dst_len = dst = 0x1234; *(DWORD*)src = 0xabcd1234; hr = IDataConvert_DataConvert(convert, DBTYPE_UI4, DBTYPE_I2, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); -todo_wine + todo_wine ok(hr == DB_E_DATAOVERFLOW, "got %08x\n", hr); ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); ok(dst_len == sizeof(dst), "got %ld\n", dst_len); @@ -827,7 +827,7 @@ todo_wine dst_len = dst = 0x1234; *(DWORD*)src = 0x1234abcd; hr = IDataConvert_DataConvert(convert, DBTYPE_UI4, DBTYPE_I2, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); -todo_wine + todo_wine ok(hr == DB_E_DATAOVERFLOW, "got %08x\n", hr); ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); ok(dst_len == sizeof(dst), "got %ld\n", dst_len); @@ -914,7 +914,7 @@ todo_wine dst_len = dst = 0x1234; *(DWORD*)src = 0xabcd1234; hr = IDataConvert_DataConvert(convert, DBTYPE_UI4, DBTYPE_UI2, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); -todo_wine + todo_wine ok(hr == DB_E_DATAOVERFLOW, "got %08x\n", hr); ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); ok(dst_len == sizeof(dst), "got %ld\n", dst_len); @@ -923,7 +923,7 @@ todo_wine dst_len = dst = 0x1234; *(DWORD*)src = 0x1234abcd; hr = IDataConvert_DataConvert(convert, DBTYPE_UI4, DBTYPE_UI2, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); -todo_wine + todo_wine ok(hr == DB_E_DATAOVERFLOW, "got %08x\n", hr); ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); ok(dst_len == sizeof(dst), "got %ld\n", dst_len); @@ -1132,7 +1132,7 @@ static void test_converttoi4(void) hr = IDataConvert_DataConvert(convert, DBTYPE_UI4, DBTYPE_I4, 0, &dst_len, src, &i4, sizeof(i4), 0, &dst_status, 0, 0, 0); ok(hr == DB_E_ERRORSOCCURRED, "got %08x\n", hr); ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); -todo_wine + todo_wine ok(broken(dst_len == sizeof(i4)) || dst_len == 0x1234 /* W2K+ */, "got %ld\n", dst_len); ok(i4 == 0x12345678, "got %08x\n", i4); @@ -2817,9 +2817,9 @@ static void test_converttoui4(void) dst_len = 0x1234; hr = IDataConvert_DataConvert(convert, DBTYPE_I2, DBTYPE_UI4, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); ok(hr == DB_E_ERRORSOCCURRED, "got %08x\n", hr); -todo_wine + todo_wine ok(dst_status == DBSTATUS_E_SIGNMISMATCH, "got %08x\n", dst_status); -todo_wine + todo_wine ok(broken(dst_len == sizeof(dst)) || dst_len == 0x1234 /* W2K+ */, "got %ld\n", dst_len); ok(dst == 0x12345678, "got %08x\n", dst); @@ -3056,9 +3056,9 @@ static void test_converttoui8(void) dst_len = 0x1234; hr = IDataConvert_DataConvert(convert, DBTYPE_I4, DBTYPE_UI8, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); ok(hr == DB_E_ERRORSOCCURRED, "got %08x\n", hr); -todo_wine + todo_wine ok(dst_status == DBSTATUS_E_SIGNMISMATCH, "got %08x\n", dst_status); -todo_wine + todo_wine ok(dst_len == 0x1234, "got %ld\n", dst_len); ok(dst.QuadPart == 0xcc, "got %d\n", (int)dst.QuadPart);

1 0

Fabian Maurer : opcservices/tests: Avoid "misleading indentation" warnings.
by Alexandre Julliard 27 Jan '22

27 Jan '22

Module: wine Branch: master Commit: f3f8619fab8036ee78220c7a722eb00bc4c532a6 URL: https://source.winehq.org/git/wine.git/?a=commit;h=f3f8619fab8036ee78220c7a… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Thu Jan 27 20:18:06 2022 +0100 opcservices/tests: Avoid "misleading indentation" warnings. Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/opcservices/tests/opcservices.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/dlls/opcservices/tests/opcservices.c b/dlls/opcservices/tests/opcservices.c index cd683b0a206..59cd3b5df30 100644 --- a/dlls/opcservices/tests/opcservices.c +++ b/dlls/opcservices/tests/opcservices.c @@ -635,7 +635,7 @@ static void test_rel_part_uri(void) hr = IOpcPartUri_GetRawUri(rel_uri, &str); ok(SUCCEEDED(hr), "Failed to get rel uri, hr %#x.\n", hr); - todo_wine_if(i == 3 || i == 4 || i == 8 || i == 9) + todo_wine_if(i == 3 || i == 4 || i == 8 || i == 9) ok(!lstrcmpW(str, rel_uriW), "%u: unexpected rel uri %s, expected %s.\n", i, wine_dbgstr_w(str), wine_dbgstr_w(rel_uriW)); SysFreeString(str); @@ -1136,7 +1136,7 @@ static void test_combine_uri(void) hr = IOpcPartUri_GetRawUri(combined_uri, &str); ok(SUCCEEDED(hr), "%u: failed to get raw uri, hr %#x.\n", i, hr); - todo_wine_if(i == 2 || i == 3) + todo_wine_if(i == 2 || i == 3) ok(!lstrcmpW(str, combinedW), "%u: unexpected uri %s.\n", i, wine_dbgstr_w(str)); SysFreeString(str); @@ -1190,7 +1190,7 @@ static void test_create_part_uri(void) hr = IOpcPartUri_GetRawUri(part_uri, &str); ok(SUCCEEDED(hr), "Failed to get raw uri, hr %#x.\n", hr); - todo_wine_if(i == 1 || i == 2 || i == 4) + todo_wine_if(i == 1 || i == 2 || i == 4) ok(!lstrcmpW(str, rawW), "%u: unexpected raw uri %s.\n", i, wine_dbgstr_w(str)); SysFreeString(str); @@ -1200,7 +1200,7 @@ static void test_create_part_uri(void) ret = FALSE; hr = IOpcPartUri_IsEqual(part_uri, uri, &ret); ok(SUCCEEDED(hr), "IsEqual failed, hr %#x.\n", hr); - todo_wine_if(i == 1 || i == 2 || i == 4) + todo_wine_if(i == 1 || i == 2 || i == 4) ok(!!ret, "%u: unexpected result %d.\n", i, ret); IOpcPartUri_Release(part_uri);

1 0

Connor McAdams : secur32: Fix trace in schannel pull_adapter function.
by Alexandre Julliard 27 Jan '22

27 Jan '22

Module: wine Branch: master Commit: 85cb1ff91ed4d48559c5500e2ccb0b15801edc05 URL: https://source.winehq.org/git/wine.git/?a=commit;h=85cb1ff91ed4d48559c5500e… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Thu Jan 27 10:20:51 2022 -0500 secur32: Fix trace in schannel pull_adapter function. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/secur32/schannel_gnutls.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/secur32/schannel_gnutls.c b/dlls/secur32/schannel_gnutls.c index 3f947870a73..b7ffa1822b5 100644 --- a/dlls/secur32/schannel_gnutls.c +++ b/dlls/secur32/schannel_gnutls.c @@ -372,7 +372,7 @@ static ssize_t pull_adapter(gnutls_transport_ptr_t transport, void *buff, size_t SIZE_T len = buff_len; char *b; - TRACE("Push %lu bytes\n", len); + TRACE("Pull %lu bytes\n", len); b = get_buffer(t, &t->in, &len); if (!b) @@ -382,7 +382,7 @@ static ssize_t pull_adapter(gnutls_transport_ptr_t transport, void *buff, size_t } memcpy(buff, b, len); t->in.offset += len; - TRACE("Wrote %lu bytes\n", len); + TRACE("Read %lu bytes\n", len); return len; }

1 0