January 2025 - Wine-GitLab - list.winehq.org

Re: [PATCH v3 0/2] MR7196: msvcp140: Fix field order in _Mtx_t. - approved
by Alexandre Julliard (＠julliard) Jan. 27, 2025

Jan. 27, 2025

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7196

1 0

Re: [PATCH v4 0/4] MR7109: Added missing functions _FCbuild, cimag, crealf and cimagf
by Alexandre Julliard (＠julliard) Jan. 27, 2025

Jan. 27, 2025

On Mon Jan 27 21:05:00 2025 +0000, Stéphane Bacri wrote: > Hello, Piotr. > Thanks for your review and comments. However, I'm unsure the issue is > there (and I have no other clue, because no window machine nor VM). I > agree that [MS > documentation](https://learn.microsoft.com/en-us/cpp/c-runtime-library/refe… > defines the signature you mention, but it is the same for _Cbuild, and I > used it as a template. I could have suspected the entries crealf(int64) > & cimagf(int64) to be the cause if it had been a 32 vs 64 bit issue. But > I don't understand why the difference is between windows & linux. > Moreover, I checked the stage log, and I see only zeroes in the failing > tests reports, which is weird, even with params size issues). > By the way, the positive thing is I figured out that > msvcr120_app/msvcr120_app.spec had to be updated. However, the current > build-linux stage seems to fail for reasons not related to the branch (I > tried to revert a commit then commit back after setting again my user > name & email, and still fails saying email not set when other stage do > not complain). Actually _Cbuild has the same problem, but it's less obvious since it will break only on non-x86 platforms. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7109#note_92977

1 0

Re: [PATCH v4 0/4] MR7109: Added missing functions _FCbuild, cimag, crealf and cimagf
by Stéphane Bacri Jan. 27, 2025

Jan. 27, 2025

Hello, Piotr. Thanks for your review and comments. However, I'm unsure the issue is there (and I have no other clue, because no window machine nor VM). I agree that [MS documentation](https://learn.microsoft.com/en-us/cpp/c-runtime-library/refe… defines the signature you mention, but it is the same for _Cbuild, and I used it as a template. I could have suspected the entries crealf(int64) & cimagf(int64) to be the cause if it had been a 32 vs 64 bit issue. But I don't understand why the difference is between windows & linux. Moreover, I checked the stage log, and I see only zeroes in the failing tests reports, which is weird, even with params size issues). By the way, the positive thing is I figured out that msvcr120_app/msvcr120_app.spec had to be updated. However, the current build-linux stage seems to fail for reasons not related to the branch (I tried to revert a commit then commit back after setting again my user name & email, and still fails saying email not set when other stage do not complain). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7109#note_92974

1 0

[PATCH v12 0/2] MR7169: ntdll: Fix the return value of NtDelayExecution.
by William Horvath (＠whrvth) Jan. 27, 2025

Jan. 27, 2025

The key change is to never return STATUS_TIMEOUT, and to instead return the result of NtYieldExecution() if zero timeout was passed, or STATUS_SUCCESS otherwise. An overview of the correct values for each combination, copied from the test commit: - Non-alertable, zero timeout: STATUS_SUCCESS or STATUS_NO_YIELD_PERFORMED - Non-alertable, non-zero timeout: STATUS_SUCCESS - Alertable, zero timeout: STATUS_SUCCESS, STATUS_NO_YIELD_PERFORMED, or STATUS_USER_APC - Alertable, non-zero timeout: STATUS_SUCCESS or STATUS_USER_APC -- v12: ntdll: Fix the return value of NtDelayExecution. ntdll/tests: Add tests for NtDelayExecution and Sleep(Ex). https://gitlab.winehq.org/wine/wine/-/merge_requests/7169

3 4

[PATCH v11 0/2] MR7169: ntdll: Fix the return value of NtDelayExecution.
by William Horvath (＠whrvth) Jan. 27, 2025

Jan. 27, 2025

The key change is to never return STATUS_TIMEOUT, and to instead return the result of NtYieldExecution() if zero timeout was passed, or STATUS_SUCCESS otherwise. An overview of the correct values for each combination, copied from the test commit: - Non-alertable, zero timeout: STATUS_SUCCESS or STATUS_NO_YIELD_PERFORMED - Non-alertable, non-zero timeout: STATUS_SUCCESS - Alertable, zero timeout: STATUS_SUCCESS, STATUS_NO_YIELD_PERFORMED, or STATUS_USER_APC - Alertable, non-zero timeout: STATUS_SUCCESS or STATUS_USER_APC -- v11: ntdll: Fix the return value of NtDelayExecution. ntdll/tests: Add tests for NtDelayExecution and Sleep(Ex). https://gitlab.winehq.org/wine/wine/-/merge_requests/7169

3 10

[PATCH v2 0/4] MR7203: dlls/oleaut32: Cleanups
by Edward OC (＠funfunctor) Jan. 27, 2025

Jan. 27, 2025

The intention is to have ICreateTypeLib2_fnDeleteTypeInfo() fully implemented such that Cashbooks can run on Linux. I believe this should fix a number of other Windows apps that lean heavily on oleaut32.dll as well. -- v2: dlls/oleaut32: Add DeleteTypeInfo() base-cases in typelib.c https://gitlab.winehq.org/wine/wine/-/merge_requests/7203

5 8

[PATCH 0/3] MR7206: Update some include headers regarding debug information.
by eric pouech (＠epo) Jan. 27, 2025

Jan. 27, 2025

Mostly adding a couple of missing bits. Regarding the 32-bit KDHELP structure definition: - I checked against some SDK:s back to 2000, and the overall size of this structure hasn't changed ever since (some fields have been added with decreasing Reserved array size) - so it's really Wine's header which were missing these fields not the callers -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7206

3 4

[PATCH v10 0/2] MR7169: ntdll: Fix the return value of NtDelayExecution.
by William Horvath (＠whrvth) Jan. 27, 2025

Jan. 27, 2025

The key change is to never return STATUS_TIMEOUT, and to instead return the result of NtYieldExecution() if zero timeout was passed, or STATUS_SUCCESS otherwise. An overview of the correct values for each combination, copied from the test commit: - Non-alertable, zero timeout: STATUS_SUCCESS or STATUS_NO_YIELD_PERFORMED - Non-alertable, non-zero timeout: STATUS_SUCCESS - Alertable, zero timeout: STATUS_SUCCESS, STATUS_NO_YIELD_PERFORMED, or STATUS_USER_APC - Alertable, non-zero timeout: STATUS_SUCCESS or STATUS_USER_APC -- v10: ntdll: Fix the return value of NtDelayExecution. ntdll/tests: Add tests for NtDelayExecution and Sleep(Ex). https://gitlab.winehq.org/wine/wine/-/merge_requests/7169

4 8

[PATCH 0/1] MR7138: sapi: Remove some unnecessary traces.
by Shaun Ren (＠shaunren) Jan. 27, 2025

Jan. 27, 2025

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/7138

3 2

[PATCH v3 0/5] MR7137: sapi: Implement most ISpStream methods.
by Shaun Ren (＠shaunren) Jan. 27, 2025

Jan. 27, 2025

Needed for Diablo 4's screen reader. -- v3: sapi: Implement IStream methods for SpStream. sapi: Implement ISpStream::GetFormat. https://gitlab.winehq.org/wine/wine/-/merge_requests/7137

4 7