- Wine-Commits - list.winehq.org

Rémi Bernon : imm32: Fix mixed-up HIMC / imc pointers in ImmTranslateMessage.
by Alexandre Julliard April 4, 2023

April 4, 2023

Module: wine Branch: master Commit: 4445fd14d707347b0bcaf2fd5be0741586f519fa URL: https://gitlab.winehq.org/wine/wine/-/commit/4445fd14d707347b0bcaf2fd5be074… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Apr 3 16:30:39 2023 +0200 imm32: Fix mixed-up HIMC / imc pointers in ImmTranslateMessage. --- dlls/imm32/imm.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/dlls/imm32/imm.c b/dlls/imm32/imm.c index 31a0b927d07..a17ffa79e73 100644 --- a/dlls/imm32/imm.c +++ b/dlls/imm32/imm.c @@ -3018,7 +3018,6 @@ BOOL WINAPI ImmTranslateMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lKeyD { struct imc *data; struct ime *ime; - HIMC imc = ImmGetContext(hwnd); BYTE state[256]; UINT scancode; TRANSMSGLIST *list = NULL; @@ -3028,8 +3027,8 @@ BOOL WINAPI ImmTranslateMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lKeyD TRACE("%p %x %x %x\n",hwnd, msg, (UINT)wParam, (UINT)lKeyData); - if (!(data = get_imc_data( imc ))) return FALSE; - if (!(ime = imc_select_ime( imc ))) return FALSE; + if (!(data = get_imc_data( ImmGetContext( hwnd ) ))) return FALSE; + if (!(ime = imc_select_ime( data ))) return FALSE; if (data->lastVK == VK_PROCESSKEY) return FALSE; GetKeyboardState(state); @@ -3051,12 +3050,12 @@ BOOL WINAPI ImmTranslateMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lKeyD else uVirtKey = data->lastVK; - msg_count = ime->pImeToAsciiEx( uVirtKey, scancode, state, list, 0, imc ); + msg_count = ime->pImeToAsciiEx( uVirtKey, scancode, state, list, 0, data->handle ); TRACE("%i messages generated\n",msg_count); if (msg_count && msg_count <= list_count) for (i = 0; i < msg_count; i++) imc_post_message( data, list->TransMsg + i ); else if (msg_count > list_count) - ImmGenerateMessage(imc); + ImmGenerateMessage( data->handle ); free( list );

1 0

Rémi Bernon : imm32: Simplify control flow in ImmTranslateMessage.
by Alexandre Julliard April 4, 2023

April 4, 2023

Module: wine Branch: master Commit: 101743205d09919452e514c506a07f1d1fb2942f URL: https://gitlab.winehq.org/wine/wine/-/commit/101743205d09919452e514c506a07f… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Apr 3 18:32:29 2023 +0200 imm32: Simplify control flow in ImmTranslateMessage. --- dlls/imm32/imm.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/dlls/imm32/imm.c b/dlls/imm32/imm.c index a17ffa79e73..4609c689ed2 100644 --- a/dlls/imm32/imm.c +++ b/dlls/imm32/imm.c @@ -3033,6 +3033,7 @@ BOOL WINAPI ImmTranslateMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lKeyD GetKeyboardState(state); scancode = lKeyData >> 0x10 & 0xff; + uVirtKey = data->lastVK; list = calloc( list_count, sizeof(TRANSMSG) + sizeof(DWORD) ); list->uMsgCount = list_count; @@ -3047,15 +3048,12 @@ BOOL WINAPI ImmTranslateMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lKeyD ToUnicodeEx(data->lastVK, scancode, state, &chr, 1, 0, GetKeyboardLayout(0)); uVirtKey = MAKELONG(data->lastVK,chr); } - else - uVirtKey = data->lastVK; msg_count = ime->pImeToAsciiEx( uVirtKey, scancode, state, list, 0, data->handle ); TRACE("%i messages generated\n",msg_count); - if (msg_count && msg_count <= list_count) - for (i = 0; i < msg_count; i++) imc_post_message( data, list->TransMsg + i ); - else if (msg_count > list_count) - ImmGenerateMessage( data->handle ); + + if (msg_count > list_count) ImmGenerateMessage( data->handle ); + else for (i = 0; i < msg_count; i++) imc_post_message( data, list->TransMsg + i ); free( list );

1 0

Rémi Bernon : imm32: Cleanup parameters and traces in ImmTranslateMessage.
by Alexandre Julliard April 4, 2023

April 4, 2023

Module: wine Branch: master Commit: 8d0e7d8cdcde897a9bf50a7047fc3bd4a503985c URL: https://gitlab.winehq.org/wine/wine/-/commit/8d0e7d8cdcde897a9bf50a7047fc3b… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Apr 3 18:23:38 2023 +0200 imm32: Cleanup parameters and traces in ImmTranslateMessage. --- dlls/imm32/imm.c | 37 ++++++++++++++++--------------------- 1 file changed, 16 insertions(+), 21 deletions(-) diff --git a/dlls/imm32/imm.c b/dlls/imm32/imm.c index 4609c689ed2..736e1dcf211 100644 --- a/dlls/imm32/imm.c +++ b/dlls/imm32/imm.c @@ -3014,52 +3014,47 @@ BOOL WINAPI ImmGenerateMessage(HIMC hIMC) * ImmTranslateMessage(IMM32.@) * ( Undocumented, call internally and from user32.dll ) */ -BOOL WINAPI ImmTranslateMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lKeyData) +BOOL WINAPI ImmTranslateMessage( HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam ) { + static const DWORD list_count = 10; + UINT scan, vkey, count, i; struct imc *data; struct ime *ime; BYTE state[256]; - UINT scancode; TRANSMSGLIST *list = NULL; - UINT msg_count, i; - UINT uVirtKey; - static const DWORD list_count = 10; + WCHAR chr; - TRACE("%p %x %x %x\n",hwnd, msg, (UINT)wParam, (UINT)lKeyData); + TRACE( "hwnd %p, msg %#x, wparam %#Ix, lparam %#Ix\n", hwnd, msg, wparam, lparam ); if (!(data = get_imc_data( ImmGetContext( hwnd ) ))) return FALSE; if (!(ime = imc_select_ime( data ))) return FALSE; if (data->lastVK == VK_PROCESSKEY) return FALSE; - GetKeyboardState(state); - scancode = lKeyData >> 0x10 & 0xff; - uVirtKey = data->lastVK; + GetKeyboardState( state ); + scan = lparam >> 0x10 & 0xff; + vkey = data->lastVK; list = calloc( list_count, sizeof(TRANSMSG) + sizeof(DWORD) ); list->uMsgCount = list_count; if (ime->info.fdwProperty & IME_PROP_KBD_CHAR_FIRST) { - WCHAR chr; - - if (!ime_is_unicode( ime )) - ToAscii( data->lastVK, scancode, state, &chr, 0 ); - else - ToUnicodeEx(data->lastVK, scancode, state, &chr, 1, 0, GetKeyboardLayout(0)); - uVirtKey = MAKELONG(data->lastVK,chr); + if (!ime_is_unicode( ime )) ToAscii( data->lastVK, scan, state, &chr, 0 ); + else ToUnicodeEx( data->lastVK, scan, state, &chr, 1, 0, GetKeyboardLayout( 0 ) ); + vkey = MAKELONG( data->lastVK, chr ); } - msg_count = ime->pImeToAsciiEx( uVirtKey, scancode, state, list, 0, data->handle ); - TRACE("%i messages generated\n",msg_count); + count = ime->pImeToAsciiEx( vkey, scan, state, list, 0, data->handle ); + TRACE( "%u messages generated\n", count ); - if (msg_count > list_count) ImmGenerateMessage( data->handle ); - else for (i = 0; i < msg_count; i++) imc_post_message( data, list->TransMsg + i ); + if (count > list_count) ImmGenerateMessage( data->handle ); + else for (i = 0; i < count; i++) imc_post_message( data, list->TransMsg + i ); free( list ); data->lastVK = VK_PROCESSKEY; - return (msg_count > 0); + return count > 0; } /***********************************************************************

1 0

Rémi Bernon : imm32: Rewrite ImmInternalPostIMEMessage helper as imc_post_message.
by Alexandre Julliard April 4, 2023

April 4, 2023

Module: wine Branch: master Commit: 3b4aa1662e79b025e46d4a10426eda504beb4c6b URL: https://gitlab.winehq.org/wine/wine/-/commit/3b4aa1662e79b025e46d4a10426eda… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Apr 3 16:47:04 2023 +0200 imm32: Rewrite ImmInternalPostIMEMessage helper as imc_post_message. --- dlls/imm32/imm.c | 21 ++++++--------------- 1 file changed, 6 insertions(+), 15 deletions(-) diff --git a/dlls/imm32/imm.c b/dlls/imm32/imm.c index e815f0c479b..de32196a133 100644 --- a/dlls/imm32/imm.c +++ b/dlls/imm32/imm.c @@ -734,14 +734,11 @@ BOOL WINAPI DllMain(HINSTANCE hInstDLL, DWORD fdwReason, LPVOID lpReserved) return TRUE; } -/* for posting messages as the IME */ -static void ImmInternalPostIMEMessage( struct imc *data, UINT msg, WPARAM wParam, LPARAM lParam ) +static void imc_post_message( struct imc *imc, TRANSMSG *message ) { - HWND target = GetFocus(); - if (!target) - PostMessageW(data->IMC.hWnd,msg,wParam,lParam); - else - PostMessageW(target, msg, wParam, lParam); + HWND target; + if (!(target = GetFocus()) && !(target = imc->IMC.hWnd)) return; + PostMessageW( target, message->message, message->wParam, message->lParam ); } static void imc_send_message( struct imc *imc, TRANSMSG *message ) @@ -3031,7 +3028,7 @@ BOOL WINAPI ImmTranslateMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lKeyD BYTE state[256]; UINT scancode; TRANSMSGLIST *list = NULL; - UINT msg_count; + UINT msg_count, i; UINT uVirtKey; static const DWORD list_count = 10; @@ -3063,13 +3060,7 @@ BOOL WINAPI ImmTranslateMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lKeyD msg_count = ime->pImeToAsciiEx( uVirtKey, scancode, state, list, 0, imc ); TRACE("%i messages generated\n",msg_count); if (msg_count && msg_count <= list_count) - { - UINT i; - LPTRANSMSG msgs = list->TransMsg; - - for (i = 0; i < msg_count; i++) - ImmInternalPostIMEMessage(data, msgs[i].message, msgs[i].wParam, msgs[i].lParam); - } + for (i = 0; i < msg_count; i++) imc_post_message( data, list->TransMsg + i ); else if (msg_count > list_count) ImmGenerateMessage(imc);

1 0

Rémi Bernon : imm32: Rewrite ImmInternalSendIMEMessage helper as imc_send_message.
by Alexandre Julliard April 4, 2023

April 4, 2023

Module: wine Branch: master Commit: f6cf1d4432b2843b789056f6e93a8c41596ef03b URL: https://gitlab.winehq.org/wine/wine/-/commit/f6cf1d4432b2843b789056f6e93a8c… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Apr 3 16:45:44 2023 +0200 imm32: Rewrite ImmInternalSendIMEMessage helper as imc_send_message. --- dlls/imm32/imm.c | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/dlls/imm32/imm.c b/dlls/imm32/imm.c index e165544ba5e..e815f0c479b 100644 --- a/dlls/imm32/imm.c +++ b/dlls/imm32/imm.c @@ -744,14 +744,11 @@ static void ImmInternalPostIMEMessage( struct imc *data, UINT msg, WPARAM wParam PostMessageW(target, msg, wParam, lParam); } -/* for sending messages as the IME */ -static void ImmInternalSendIMEMessage( struct imc *data, UINT msg, WPARAM wParam, LPARAM lParam ) +static void imc_send_message( struct imc *imc, TRANSMSG *message ) { - HWND target = GetFocus(); - if (!target) - SendMessageW(data->IMC.hWnd,msg,wParam,lParam); - else - SendMessageW(target, msg, wParam, lParam); + HWND target; + if (!(target = GetFocus()) && !(target = imc->IMC.hWnd)) return; + SendMessageW( target, message->message, message->wParam, message->lParam ); } static LRESULT ImmInternalSendIMENotify( struct imc *data, WPARAM notify, LPARAM lParam ) @@ -3014,9 +3011,7 @@ BOOL WINAPI ImmGenerateMessage(HIMC hIMC) data->IMC.dwNumMsgBuf = 0; lpTransMsg = ImmLockIMCC(hMsgBuf); - for (i = 0; i < dwNumMsgBuf; i++) - ImmInternalSendIMEMessage(data, lpTransMsg[i].message, lpTransMsg[i].wParam, lpTransMsg[i].lParam); - + for (i = 0; i < dwNumMsgBuf; i++) imc_send_message( data, lpTransMsg + i ); ImmUnlockIMCC(hMsgBuf); ImmDestroyIMCC(hMsgBuf); }

1 0

Alex Henrie : wldap32: Skip over the TLS certificate check if not using TLS.
by Alexandre Julliard April 4, 2023

April 4, 2023

Module: wine Branch: master Commit: 12b5da66e9b7ead4e0f09b5880988e7a4a78610e URL: https://gitlab.winehq.org/wine/wine/-/commit/12b5da66e9b7ead4e0f09b5880988e… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Mon Apr 3 23:42:27 2023 -0600 wldap32: Skip over the TLS certificate check if not using TLS. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54772 --- dlls/wldap32/init.c | 20 ++++++++++---------- dlls/wldap32/tests/parse.c | 2 ++ 2 files changed, 12 insertions(+), 10 deletions(-) diff --git a/dlls/wldap32/init.c b/dlls/wldap32/init.c index e68782add8d..f1fd33cd34e 100644 --- a/dlls/wldap32/init.c +++ b/dlls/wldap32/init.c @@ -271,17 +271,17 @@ ULONG CDECL WLDAP32_ldap_connect( LDAP *ld, struct l_timeval *timeout ) if ((ret = ldap_get_option( CTX(ld), LDAP_OPT_X_TLS_SSL_CTX, &tls_context ))) return map_error( ret ); - if (QueryContextAttributesA( tls_context, SECPKG_ATTR_REMOTE_CERT_CONTEXT, &cert ) != SEC_E_OK) - return WLDAP32_LDAP_SERVER_DOWN; - - if (cert_callback( ld, &cert )) - { - TRACE( "accepted\n" ); - } - else + if (QueryContextAttributesA( tls_context, SECPKG_ATTR_REMOTE_CERT_CONTEXT, &cert ) == SEC_E_OK) { - WARN( "rejected\n" ); - return WLDAP32_LDAP_SERVER_DOWN; + if (cert_callback( ld, &cert )) + { + TRACE( "accepted\n" ); + } + else + { + WARN( "rejected\n" ); + return WLDAP32_LDAP_SERVER_DOWN; + } } } diff --git a/dlls/wldap32/tests/parse.c b/dlls/wldap32/tests/parse.c index 15a24716305..995297cce10 100644 --- a/dlls/wldap32/tests/parse.c +++ b/dlls/wldap32/tests/parse.c @@ -586,6 +586,8 @@ static void test_opt_server_certificate(void) ok( !ret, "ldap_set_optionA should succeed, got %#lx\n", ret ); ret = ldap_set_optionA( ld, LDAP_OPT_SERVER_CERTIFICATE, &verify_certificate ); ok( !ret, "ldap_set_optionA should succeed, got %#lx\n", ret ); + ret = ldap_connect( ld, NULL ); + ok( !ret, "ldap_connect should succeed, got %#lx\n", ret ); ret = ldap_start_tls_sA( ld, NULL, NULL, NULL, NULL ); ok( ret == LDAP_LOCAL_ERROR, "ldap_start_tls_sA should fail, got %#lx\n", ret ); ldap_unbind( ld );

1 0

Alexandre Julliard : msvcrt: Use the fmin()/fminf() implementation from the bundled musl library.
by Alexandre Julliard April 4, 2023

April 4, 2023

Module: wine Branch: master Commit: 5564512b09483156229f1231e75fc996c78d357e URL: https://gitlab.winehq.org/wine/wine/-/commit/5564512b09483156229f1231e75fc9… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Apr 3 15:36:57 2023 +0200 msvcrt: Use the fmin()/fminf() implementation from the bundled musl library. --- dlls/msvcrt/math.c | 28 ---------------------------- 1 file changed, 28 deletions(-) diff --git a/dlls/msvcrt/math.c b/dlls/msvcrt/math.c index aa33f03106a..6e29972db3e 100644 --- a/dlls/msvcrt/math.c +++ b/dlls/msvcrt/math.c @@ -8920,34 +8920,6 @@ int CDECL _fdpcomp(float x, float y) return _dpcomp(x, y); } -/********************************************************************* - * fminf (MSVCR120.@) - */ -float CDECL fminf(float x, float y) -{ - if(isnan(x)) - return y; - if(isnan(y)) - return x; - if(x==0 && y==0) - return signbit(x) ? x : y; - return x<y ? x : y; -} - -/********************************************************************* - * fmin (MSVCR120.@) - */ -double CDECL fmin(double x, double y) -{ - if(isnan(x)) - return y; - if(isnan(y)) - return x; - if(x==0 && y==0) - return signbit(x) ? x : y; - return x<y ? x : y; -} - /********************************************************************* * asinh (MSVCR120.@) *

1 0

Alexandre Julliard : msvcrt: Use the fdim()/fdimf() implementation from the bundled musl library.
by Alexandre Julliard April 4, 2023

April 4, 2023

Module: wine Branch: master Commit: 233eea6cfa59f43050ada582168fffb6587f59e7 URL: https://gitlab.winehq.org/wine/wine/-/commit/233eea6cfa59f43050ada582168fff… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Apr 3 15:37:21 2023 +0200 msvcrt: Use the fdim()/fdimf() implementation from the bundled musl library. --- dlls/msvcrt/math.c | 24 ------------------------ 1 file changed, 24 deletions(-) diff --git a/dlls/msvcrt/math.c b/dlls/msvcrt/math.c index 6e29972db3e..6dcd9b5ec5f 100644 --- a/dlls/msvcrt/math.c +++ b/dlls/msvcrt/math.c @@ -8857,30 +8857,6 @@ float CDECL erfcf(float x) return FLT_MIN * FLT_MIN; } -/********************************************************************* - * fdimf (MSVCR120.@) - */ -float CDECL fdimf(float x, float y) -{ - if(isnan(x)) - return x; - if(isnan(y)) - return y; - return x>y ? x-y : 0; -} - -/********************************************************************* - * fdim (MSVCR120.@) - */ -double CDECL fdim(double x, double y) -{ - if(isnan(x)) - return x; - if(isnan(y)) - return y; - return x>y ? x-y : 0; -} - /********************************************************************* * _fdsign (MSVCR120.@) */

1 0

Alexandre Julliard : msvcrt: Use the copysign()/copysignf() implementation from the bundled musl library.
by Alexandre Julliard April 4, 2023

April 4, 2023

Module: wine Branch: master Commit: b5ba4baa516d556dd602382a2f7ece643a2e8049 URL: https://gitlab.winehq.org/wine/wine/-/commit/b5ba4baa516d556dd602382a2f7ece… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Apr 3 15:38:32 2023 +0200 msvcrt: Use the copysign()/copysignf() implementation from the bundled musl library. --- dlls/crtdll/crtdll.spec | 2 +- dlls/msvcr100/msvcr100.spec | 4 ++-- dlls/msvcr110/msvcr110.spec | 4 ++-- dlls/msvcr120/msvcr120.spec | 10 +++++----- dlls/msvcr70/msvcr70.spec | 2 +- dlls/msvcr71/msvcr71.spec | 2 +- dlls/msvcr80/msvcr80.spec | 4 ++-- dlls/msvcr90/msvcr90.spec | 4 ++-- dlls/msvcrt/math.c | 26 -------------------------- dlls/msvcrt/msvcrt.spec | 4 ++-- dlls/msvcrtd/msvcrtd.spec | 2 +- dlls/ucrtbase/ucrtbase.spec | 10 +++++----- 12 files changed, 24 insertions(+), 50 deletions(-) diff --git a/dlls/crtdll/crtdll.spec b/dlls/crtdll/crtdll.spec index d5192ad7569..c66cea5bade 100644 --- a/dlls/crtdll/crtdll.spec +++ b/dlls/crtdll/crtdll.spec @@ -64,7 +64,7 @@ @ extern _commode_dll MSVCRT__commode @ cdecl _control87(long long) @ cdecl _controlfp(long long) -@ cdecl _copysign(double double) +@ cdecl _copysign(double double) copysign @ varargs _cprintf(str) @ extern _cpumode_dll CRTDLL__cpumode_dll @ cdecl _cputs(str) diff --git a/dlls/msvcr100/msvcr100.spec b/dlls/msvcr100/msvcr100.spec index 3e2839b3908..a42fe349cf5 100644 --- a/dlls/msvcr100/msvcr100.spec +++ b/dlls/msvcr100/msvcr100.spec @@ -733,8 +733,8 @@ @ cdecl _control87(long long) @ cdecl _controlfp(long long) @ cdecl _controlfp_s(ptr long long) -@ cdecl _copysign(double double) -@ cdecl -arch=!i386 _copysignf(float float) +@ cdecl _copysign(double double) copysign +@ cdecl -arch=!i386 _copysignf(float float) copysignf @ varargs _cprintf(str) @ stub _cprintf_l @ stub _cprintf_p diff --git a/dlls/msvcr110/msvcr110.spec b/dlls/msvcr110/msvcr110.spec index c23293ea658..2bccd8c6fe9 100644 --- a/dlls/msvcr110/msvcr110.spec +++ b/dlls/msvcr110/msvcr110.spec @@ -1081,8 +1081,8 @@ @ cdecl _control87(long long) @ cdecl _controlfp(long long) @ cdecl _controlfp_s(ptr long long) -@ cdecl _copysign(double double) -@ cdecl -arch=!i386 _copysignf(float float) +@ cdecl _copysign(double double) copysign +@ cdecl -arch=!i386 _copysignf(float float) copysignf @ varargs _cprintf(str) @ stub _cprintf_l @ stub _cprintf_p diff --git a/dlls/msvcr120/msvcr120.spec b/dlls/msvcr120/msvcr120.spec index 6eac1937227..ed0f69b49bc 100644 --- a/dlls/msvcr120/msvcr120.spec +++ b/dlls/msvcr120/msvcr120.spec @@ -1073,8 +1073,8 @@ @ cdecl _control87(long long) @ cdecl _controlfp(long long) @ cdecl _controlfp_s(ptr long long) -@ cdecl _copysign(double double) -@ cdecl _copysignf(float float) +@ cdecl _copysign(double double) copysign +@ cdecl _copysignf(float float) copysignf @ varargs _cprintf(str) @ stub _cprintf_l @ stub _cprintf_p @@ -2091,9 +2091,9 @@ @ stub conj @ stub conjf @ stub conjl -@ cdecl copysign(double double) _copysign -@ cdecl copysignf(float float) _copysignf -@ cdecl copysignl(double double) _copysign +@ cdecl copysign(double double) +@ cdecl copysignf(float float) +@ cdecl copysignl(double double) copysign @ cdecl cos(double) @ cdecl -arch=!i386 cosf(float) @ cdecl cosh(double) diff --git a/dlls/msvcr70/msvcr70.spec b/dlls/msvcr70/msvcr70.spec index 60fa9ed8693..cb3a92cf395 100644 --- a/dlls/msvcr70/msvcr70.spec +++ b/dlls/msvcr70/msvcr70.spec @@ -258,7 +258,7 @@ @ extern _commode MSVCRT__commode @ cdecl _control87(long long) @ cdecl _controlfp(long long) -@ cdecl _copysign(double double) +@ cdecl _copysign(double double) copysign @ varargs _cprintf(str) @ cdecl _cputs(str) @ cdecl _cputws(wstr) diff --git a/dlls/msvcr71/msvcr71.spec b/dlls/msvcr71/msvcr71.spec index 1aacd768d46..a559b5cc376 100644 --- a/dlls/msvcr71/msvcr71.spec +++ b/dlls/msvcr71/msvcr71.spec @@ -253,7 +253,7 @@ @ extern _commode MSVCRT__commode @ cdecl _control87(long long) @ cdecl _controlfp(long long) -@ cdecl _copysign(double double) +@ cdecl _copysign(double double) copysign @ varargs _cprintf(str) @ cdecl _cputs(str) @ cdecl _cputws(wstr) diff --git a/dlls/msvcr80/msvcr80.spec b/dlls/msvcr80/msvcr80.spec index cf885153e7a..06d0bdf82cd 100644 --- a/dlls/msvcr80/msvcr80.spec +++ b/dlls/msvcr80/msvcr80.spec @@ -397,8 +397,8 @@ @ cdecl _control87(long long) @ cdecl _controlfp(long long) @ cdecl _controlfp_s(ptr long long) -@ cdecl _copysign(double double) -@ cdecl -arch=!i386 _copysignf(float float) +@ cdecl _copysign(double double) copysign +@ cdecl -arch=!i386 _copysignf(float float) copysignf @ varargs _cprintf(str) @ stub _cprintf_l @ stub _cprintf_p diff --git a/dlls/msvcr90/msvcr90.spec b/dlls/msvcr90/msvcr90.spec index 87b31d1a048..d020de98a02 100644 --- a/dlls/msvcr90/msvcr90.spec +++ b/dlls/msvcr90/msvcr90.spec @@ -380,8 +380,8 @@ @ cdecl _control87(long long) @ cdecl _controlfp(long long) @ cdecl _controlfp_s(ptr long long) -@ cdecl _copysign(double double) -@ cdecl -arch=!i386 _copysignf(float float) +@ cdecl _copysign(double double) copysign +@ cdecl -arch=!i386 _copysignf(float float) copysignf @ varargs _cprintf(str) @ stub _cprintf_l @ stub _cprintf_p diff --git a/dlls/msvcrt/math.c b/dlls/msvcrt/math.c index 6dcd9b5ec5f..c0d9baab361 100644 --- a/dlls/msvcrt/math.c +++ b/dlls/msvcrt/math.c @@ -216,19 +216,6 @@ float CDECL _chgsignf( float num ) return u.f; } -/********************************************************************* - * _copysignf (MSVCRT.@) - * - * Copied from musl: src/math/copysignf.c - */ -float CDECL _copysignf( float x, float y ) -{ - union { float f; UINT32 i; } ux = { x }, uy = { y }; - ux.i &= 0x7fffffff; - ux.i |= uy.i & 0x80000000; - return ux.f; -} - /********************************************************************* * _nextafterf (MSVCRT.@) * @@ -5723,19 +5710,6 @@ int CDECL fesetround(int round_mode) #endif /* _MSVCR_VER>=120 */ -/********************************************************************* - * _copysign (MSVCRT.@) - * - * Copied from musl: src/math/copysign.c - */ -double CDECL _copysign( double x, double y ) -{ - union { double f; UINT64 i; } ux = { x }, uy = { y }; - ux.i &= ~0ull >> 1; - ux.i |= uy.i & 1ull << 63; - return ux.f; -} - /********************************************************************* * _finite (MSVCRT.@) */ diff --git a/dlls/msvcrt/msvcrt.spec b/dlls/msvcrt/msvcrt.spec index 6dcb0767c31..43259d7abad 100644 --- a/dlls/msvcrt/msvcrt.spec +++ b/dlls/msvcrt/msvcrt.spec @@ -367,8 +367,8 @@ @ cdecl _control87(long long) @ cdecl _controlfp(long long) @ cdecl _controlfp_s(ptr long long) -@ cdecl _copysign(double double) -@ cdecl -arch=!i386 _copysignf(float float) +@ cdecl _copysign(double double) copysign +@ cdecl -arch=!i386 _copysignf(float float) copysignf @ varargs _cprintf(str) # stub _cprintf_l(str ptr) # stub _cprintf_p(str) diff --git a/dlls/msvcrtd/msvcrtd.spec b/dlls/msvcrtd/msvcrtd.spec index 33696eba829..5cb09d92c74 100644 --- a/dlls/msvcrtd/msvcrtd.spec +++ b/dlls/msvcrtd/msvcrtd.spec @@ -242,7 +242,7 @@ @ extern _commode MSVCRT__commode @ cdecl _control87(long long) @ cdecl _controlfp(long long) -@ cdecl _copysign(double double) +@ cdecl _copysign(double double) copysign @ varargs _cprintf(str) @ cdecl _cputs(str) @ cdecl _creat(str long) diff --git a/dlls/ucrtbase/ucrtbase.spec b/dlls/ucrtbase/ucrtbase.spec index bc9dd2bcbc6..87a266198bd 100644 --- a/dlls/ucrtbase/ucrtbase.spec +++ b/dlls/ucrtbase/ucrtbase.spec @@ -237,8 +237,8 @@ @ cdecl _control87(long long) @ cdecl _controlfp(long long) @ cdecl _controlfp_s(ptr long long) -@ cdecl _copysign(double double) -@ cdecl _copysignf(float float) +@ cdecl _copysign(double double) copysign +@ cdecl _copysignf(float float) copysignf @ cdecl _cputs(str) @ cdecl _cputws(wstr) @ cdecl _creat(str long) @@ -2236,9 +2236,9 @@ @ stub conj @ stub conjf @ stub conjl -@ cdecl copysign(double double) _copysign -@ cdecl copysignf(float float) _copysignf -@ cdecl copysignl(double double) _copysign +@ cdecl copysign(double double) +@ cdecl copysignf(float float) +@ cdecl copysignl(double double) copysign @ cdecl cos(double) @ cdecl -arch=!i386 cosf(float) @ cdecl cosh(double)

1 0

Alexandre Julliard : msvcrt: Use the cbrt()/cbrtf() implementation from the bundled musl library.
by Alexandre Julliard April 4, 2023

April 4, 2023

Module: wine Branch: master Commit: 406c583c3bdb369e6faef763f3c77ce0bfe0670a URL: https://gitlab.winehq.org/wine/wine/-/commit/406c583c3bdb369e6faef763f3c77c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Apr 3 15:33:29 2023 +0200 msvcrt: Use the cbrt()/cbrtf() implementation from the bundled musl library. --- dlls/msvcrt/math.c | 84 ------------------------------------------------------ 1 file changed, 84 deletions(-) diff --git a/dlls/msvcrt/math.c b/dlls/msvcrt/math.c index 4d8e0e17b6f..ef5d194eab8 100644 --- a/dlls/msvcrt/math.c +++ b/dlls/msvcrt/math.c @@ -7722,90 +7722,6 @@ void __cdecl __libm_sse2_sqrt_precise(void) #if _MSVCR_VER>=120 -/********************************************************************* - * cbrt (MSVCR120.@) - * - * Copied from musl: src/math/cbrt.c - */ -double CDECL cbrt(double x) -{ - static const UINT32 B1 = 715094163, B2 = 696219795; - static const double P0 = 1.87595182427177009643, - P1 = -1.88497979543377169875, - P2 = 1.621429720105354466140, - P3 = -0.758397934778766047437, - P4 = 0.145996192886612446982; - - union {double f; UINT64 i;} u = {x}; - double r,s,t,w; - UINT32 hx = u.i >> 32 & 0x7fffffff; - - if (hx >= 0x7ff00000) /* cbrt(NaN,INF) is itself */ - return x + x; - - if (hx < 0x00100000) { /* zero or subnormal? */ - u.f = x * 0x1p54; - hx = u.i>>32 & 0x7fffffff; - if (hx == 0) - return x; - hx = hx / 3 + B2; - } else - hx = hx / 3 + B1; - u.i &= 1ULL << 63; - u.i |= (UINT64)hx << 32; - t = u.f; - - r = (t * t) * (t / x); - t = t * ((P0 + r * (P1 + r * P2)) + ((r * r) * r) * (P3 + r * P4)); - - u.f = t; - u.i = (u.i + 0x80000000) & 0xffffffffc0000000ULL; - t = u.f; - - s = t * t; - r = x / s; - w = t + t; - r = (r - t) / (w + r); - t = t + t * r; - return t; -} - -/********************************************************************* - * cbrtf (MSVCR120.@) - * - * Copied from musl: src/math/cbrtf.c - */ -float CDECL cbrtf(float x) -{ - static const unsigned B1 = 709958130, B2 = 642849266; - - double r,T; - union {float f; UINT32 i;} u = {x}; - UINT32 hx = u.i & 0x7fffffff; - - if (hx >= 0x7f800000) - return x + x; - - if (hx < 0x00800000) { /* zero or subnormal? */ - if (hx == 0) - return x; - u.f = x * 0x1p24f; - hx = u.i & 0x7fffffff; - hx = hx / 3 + B2; - } else - hx = hx / 3 + B1; - u.i &= 0x80000000; - u.i |= hx; - - T = u.f; - r = T * T * T; - T = T * (x + x + r) / (x + r + r); - - r = T * T * T; - T = T * (x + x + r) / (x + r + r); - return T; -} - /********************************************************************* * exp2 (MSVCR120.@) *

1 0