October 2022 - Wine-Commits

Francois Gouget : dbghelp: Fix the spelling of a couple of comments.
by Alexandre Julliard Oct. 22, 2022

Oct. 22, 2022

Module: wine Branch: master Commit: 0eb40cdbd07c51a70d54e3ebbd624769964a69ea URL: https://gitlab.winehq.org/wine/wine/-/commit/0eb40cdbd07c51a70d54e3ebbd6247… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Oct 20 16:23:05 2022 +0200 dbghelp: Fix the spelling of a couple of comments. --- dlls/dbghelp/dbghelp_private.h | 4 ++-- dlls/dbghelp/dwarf.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/dlls/dbghelp/dbghelp_private.h b/dlls/dbghelp/dbghelp_private.h index a5ea41ae014..5e63d949a43 100644 --- a/dlls/dbghelp/dbghelp_private.h +++ b/dlls/dbghelp/dbghelp_private.h @@ -271,11 +271,11 @@ struct symt_data * address in one of its ranges * * Notes: - * (A): shall evolve but storage is native is awkward: from PGO testing, the + * (A): shall evolve but storage in native is awkward: from PGO testing, the * top function is stored with its first range of address; all the others * are stored as blocks, children of compiland, but which lexical parent * is the top function. This breaks the natural assumption that - * children <> lexical parent is symetrical. + * children <> lexical parent is symmetrical. * (B): see dwarf.c for some gory discrepancies between native & builtin * DbgHelp. */ diff --git a/dlls/dbghelp/dwarf.c b/dlls/dbghelp/dwarf.c index f8b3f8d717c..f9ffa374dbc 100644 --- a/dlls/dbghelp/dwarf.c +++ b/dlls/dbghelp/dwarf.c @@ -2201,7 +2201,7 @@ static void dwarf2_parse_subprogram_block(dwarf2_subprogram_t* subpgm, } /* Dwarf tends to keep the structure of the C/C++ program, and emits DW_TAG_lexical_block - * for every block in source program. + * for every block the in source program. * With inlining and other optimizations, code for a block no longer lies in a contiguous * chunk of memory. It's hence described with (potentially) multiple chunks of memory. * Then each variable of each block is attached to its block. (A) @@ -2217,7 +2217,7 @@ static void dwarf2_parse_subprogram_block(dwarf2_subprogram_t* subpgm, * * DbgHelp only exposes a contiguous chunk of memory for a block. * - * => Store internaly all the ranges of addresses in a block, but only expose the size + * => Store internally all the ranges of addresses in a block, but only expose the size * of the first chunk of memory. * Otherwise, it would break the rule: blocks' chunks don't overlap. * Note: This could mislead some programs using the blocks' address and size information.

1 0

Francois Gouget : rpcrt4/tests: Fix the spelling of a couple of comments.
by Alexandre Julliard Oct. 22, 2022

Oct. 22, 2022

Module: wine Branch: master Commit: e7be6ae2f1d85c380098350cff3ed1c2fb25e5bc URL: https://gitlab.winehq.org/wine/wine/-/commit/e7be6ae2f1d85c380098350cff3ed1… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Oct 20 16:23:31 2022 +0200 rpcrt4/tests: Fix the spelling of a couple of comments. --- dlls/rpcrt4/tests/cstub.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/rpcrt4/tests/cstub.c b/dlls/rpcrt4/tests/cstub.c index d44ee36088c..794d85744ef 100644 --- a/dlls/rpcrt4/tests/cstub.c +++ b/dlls/rpcrt4/tests/cstub.c @@ -1534,7 +1534,7 @@ static void test_ChannelBufferRefCount(IPSFactoryBuffer *ppsf) * so it's possible these are freed out from under it. * E.g. an event sink might unadvise upon receiving the event it was waiting for; * this unadvise could be reentrant to Invoke because SendReceive pumps STA messages. - * The source would then erase that conection point entry and Release the proxy. */ + * The source would then erase that connection point entry and Release the proxy. */ IRpcProxyBuffer_Disconnect(proxy_buffer); ok(test_chanbuf.RefCount == 1, "got %ld\n", test_chanbuf.RefCount); IRpcProxyBuffer_Release(proxy_buffer); @@ -1542,7 +1542,7 @@ static void test_ChannelBufferRefCount(IPSFactoryBuffer *ppsf) ok(refs == 0, "got %ld\n", refs); ok(test_chanbuf.RefCount == 1, "got %ld\n", test_chanbuf.RefCount); - /* NdrProxyFreeBuffer must not dereference the the now-freed proxy_if1, + /* NdrProxyFreeBuffer must not dereference the now-freed proxy_if1, * yet should still free the remaining reference on test_chanbuf */ NdrProxyFreeBuffer(proxy_if1, &stubMessage); ok(test_chanbuf.RefCount == 0, "got %ld\n", test_chanbuf.RefCount);

1 0

Jinoh Kang : win32u: Fix data race in NtUserGetProcessDpiAwarenessContext.
by Alexandre Julliard Oct. 22, 2022

Oct. 22, 2022

Module: wine Branch: master Commit: 25b05840d4fad5d12f308077925e3cf92bcebfb7 URL: https://gitlab.winehq.org/wine/wine/-/commit/25b05840d4fad5d12f308077925e3c… Author: Jinoh Kang <jinoh.kang.kr(a)gmail.com> Date: Mon Sep 19 00:13:24 2022 +0900 win32u: Fix data race in NtUserGetProcessDpiAwarenessContext. --- dlls/win32u/sysparams.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 498ed386386..9f0812b843e 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -5330,14 +5330,17 @@ BOOL WINAPI NtUserSetProcessDpiAwarenessContext( ULONG awareness, ULONG unknown */ ULONG WINAPI NtUserGetProcessDpiAwarenessContext( HANDLE process ) { + DPI_AWARENESS val; + if (process && process != GetCurrentProcess()) { WARN( "not supported on other process %p\n", process ); return NTUSER_DPI_UNAWARE; } - if (!dpi_awareness) return NTUSER_DPI_UNAWARE; - return dpi_awareness; + val = ReadNoFence( &dpi_awareness ); + if (!val) return NTUSER_DPI_UNAWARE; + return val; } BOOL message_beep( UINT i )

1 0

Francois Gouget : ddraw/tests: Mark tests failing randomly on Windows 8 as flaky.
by Alexandre Julliard Oct. 22, 2022

Oct. 22, 2022

Module: wine Branch: master Commit: c2adce37f4b9d9cfc7fb0d08ff60344d4d4a0ff3 URL: https://gitlab.winehq.org/wine/wine/-/commit/c2adce37f4b9d9cfc7fb0d08ff6034… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Sat Oct 1 08:45:16 2022 +0200 ddraw/tests: Mark tests failing randomly on Windows 8 as flaky. Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=53134 --- dlls/ddraw/tests/ddraw1.c | 18 ++++++++++++++++++ dlls/ddraw/tests/ddraw2.c | 18 ++++++++++++++++++ dlls/ddraw/tests/ddraw4.c | 18 ++++++++++++++++++ dlls/ddraw/tests/ddraw7.c | 18 ++++++++++++++++++ 4 files changed, 72 insertions(+)

1 0

Francois Gouget : msdasql: Fix a typo in the is_fixed_length() function name.
by Alexandre Julliard Oct. 22, 2022

Oct. 22, 2022

Module: wine Branch: master Commit: a00a64b9c092a39d26e2016e98b358c1c29553f4 URL: https://gitlab.winehq.org/wine/wine/-/commit/a00a64b9c092a39d26e2016e98b358… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Oct 20 16:23:53 2022 +0200 msdasql: Fix a typo in the is_fixed_length() function name. --- dlls/msdasql/session.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/msdasql/session.c b/dlls/msdasql/session.c index 27d952276e5..67dc30bf09e 100644 --- a/dlls/msdasql/session.c +++ b/dlls/msdasql/session.c @@ -263,7 +263,7 @@ static BOOL is_variable_length(SQLSMALLINT type) return FALSE; } -static BOOL is_fixed_legnth(SQLSMALLINT type) +static BOOL is_fixed_length(SQLSMALLINT type) { switch(type) { @@ -1059,7 +1059,7 @@ static HRESULT WINAPI rowset_colsinfo_GetColumnInfo(IColumnsInfo *iface, DBORDIN if (ColumnDataNullable) dbcolumn[i].dwFlags |= DBCOLUMNFLAGS_ISNULLABLE | DBCOLUMNFLAGS_MAYBENULL; - if (is_fixed_legnth(ColumnDataType)) + if (is_fixed_length(ColumnDataType)) dbcolumn[i].dwFlags |= DBCOLUMNFLAGS_ISFIXEDLENGTH; ret = SQLColAttribute(rowset->hstmt, i+1, SQL_DESC_SCALE, NULL, 0, NULL, &length);

1 0

Francois Gouget : d3d10core/tests: Fix the spelling of a comment.
by Alexandre Julliard Oct. 22, 2022

Oct. 22, 2022

Module: wine Branch: master Commit: 5f0b7390e99fb5b26a78ee5d99266d75c810baad URL: https://gitlab.winehq.org/wine/wine/-/commit/5f0b7390e99fb5b26a78ee5d99266d… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Oct 20 16:22:23 2022 +0200 d3d10core/tests: Fix the spelling of a comment. --- dlls/d3d10core/tests/d3d10core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/d3d10core/tests/d3d10core.c b/dlls/d3d10core/tests/d3d10core.c index 74f689630eb..cd14371197f 100644 --- a/dlls/d3d10core/tests/d3d10core.c +++ b/dlls/d3d10core/tests/d3d10core.c @@ -4813,7 +4813,7 @@ static void test_pipeline_statistics_query(void) get_query_data(query, &data, sizeof(data)); /* WARP devices randomly return all-zeroed structures as if the draw did not happen. Flushing and - * sleeping a second before ending the query reduces the likelyhood of hitting the bug a lot, but + * sleeping a second before ending the query reduces the likelihood of hitting the bug a lot, but * does not eliminate it entirely. To make things work reliably ignore such broken results. */ if (is_warp_device(device) && !memcmp(&data, &zero_data, sizeof(data))) {

1 0

Alex Henrie : msvcrt: Change return type of _putenv_s to errno_t.
by Alexandre Julliard Oct. 22, 2022

Oct. 22, 2022

Module: wine Branch: master Commit: 1f657729e5ef6b38a6368afd462105cd9c0e2715 URL: https://gitlab.winehq.org/wine/wine/-/commit/1f657729e5ef6b38a6368afd462105… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Thu Oct 20 08:56:25 2022 -0600 msvcrt: Change return type of _putenv_s to errno_t. --- dlls/msvcrt/environ.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/msvcrt/environ.c b/dlls/msvcrt/environ.c index 698dc761381..83a493c1a9a 100644 --- a/dlls/msvcrt/environ.c +++ b/dlls/msvcrt/environ.c @@ -163,7 +163,7 @@ finish: /********************************************************************* * _putenv_s (MSVCRT.@) */ -int CDECL _putenv_s(const char *name, const char *value) +errno_t CDECL _putenv_s(const char *name, const char *value) { int ret;

1 0

Alex Henrie : include: Add _putenv_s.
by Alexandre Julliard Oct. 22, 2022

Oct. 22, 2022

Module: wine Branch: master Commit: 7d85693750b2eae2828c5c62f9da1a3751df0184 URL: https://gitlab.winehq.org/wine/wine/-/commit/7d85693750b2eae2828c5c62f9da1a… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Thu Oct 20 08:56:25 2022 -0600 include: Add _putenv_s. Needed to compile Tera Term. --- include/msvcrt/stdlib.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/msvcrt/stdlib.h b/include/msvcrt/stdlib.h index a6e1c7eee02..c608344b115 100644 --- a/include/msvcrt/stdlib.h +++ b/include/msvcrt/stdlib.h @@ -178,6 +178,7 @@ _ACRTIMP int __cdecl _makepath_s(char*,size_t,const char*,const char*, _ACRTIMP size_t __cdecl _mbstrlen(const char*); _ACRTIMP _onexit_t __cdecl _onexit(_onexit_t); _ACRTIMP int __cdecl _putenv(const char*); +_ACRTIMP errno_t __cdecl _putenv_s(const char*,const char*); #ifndef _rotl _ACRTIMP unsigned int __cdecl _rotl(unsigned int,int); #endif

1 0

Zhiyi Zhang : comctl32/tests: Use a toolbar with TBSTYLE_FLAT in toolbar visual tests.
by Alexandre Julliard Oct. 22, 2022

Oct. 22, 2022

Module: wine Branch: master Commit: 4277b47f3e4446517817da6927f0744767b7f202 URL: https://gitlab.winehq.org/wine/wine/-/commit/4277b47f3e4446517817da6927f074… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Wed Oct 19 23:46:06 2022 +0800 comctl32/tests: Use a toolbar with TBSTYLE_FLAT in toolbar visual tests. Fix a test failure when theming is off. A toolbar without TBSTYLE_FLAT will fill its checked button background rectangle when theming is inactive, overwriting the checked pattern required for the test. --- dlls/comctl32/tests/toolbar.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dlls/comctl32/tests/toolbar.c b/dlls/comctl32/tests/toolbar.c index f66094a0b3a..7c886e1f88f 100644 --- a/dlls/comctl32/tests/toolbar.c +++ b/dlls/comctl32/tests/toolbar.c @@ -2646,7 +2646,8 @@ static void test_visual(void) RECT rect; BOOL ret; - toolbar = CreateWindowA(TOOLBARCLASSNAMEA, "", WS_CHILD | WS_VISIBLE, 0, 0, 50, 50, hMainWnd, 0, 0, NULL); + toolbar = CreateWindowA(TOOLBARCLASSNAMEA, "", WS_CHILD | WS_VISIBLE | TBSTYLE_FLAT, 0, 0, 50, + 50, hMainWnd, 0, 0, NULL); ok(!!toolbar, "Failed to create a toolbar window.\n"); /* Test that the comctl32 55AA pattern brush is not used to draw checked background when theming is on */

1 0

Brendan Shanks : xinput1_3: Set thread name for hid_update_thread_proc.
by Alexandre Julliard Oct. 22, 2022

Oct. 22, 2022

Module: wine Branch: master Commit: 480da372b25b9a432129859b9facacc6436a5a30 URL: https://gitlab.winehq.org/wine/wine/-/commit/480da372b25b9a432129859b9facac… Author: Brendan Shanks <bshanks(a)codeweavers.com> Date: Tue Oct 18 11:01:34 2022 -0700 xinput1_3: Set thread name for hid_update_thread_proc. --- dlls/xinput1_3/main.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/dlls/xinput1_3/main.c b/dlls/xinput1_3/main.c index 112fa5b436d..2abf33a8d45 100644 --- a/dlls/xinput1_3/main.c +++ b/dlls/xinput1_3/main.c @@ -707,6 +707,8 @@ static DWORD WINAPI hid_update_thread_proc(void *param) HWND hwnd; MSG msg; + SetThreadDescription(GetCurrentThread(), L"wine_xinput_hid_update"); + RegisterClassExW(&cls); hwnd = CreateWindowExW(0, cls.lpszClassName, NULL, 0, 0, 0, 0, 0, HWND_MESSAGE, NULL, NULL, NULL);

1 0