- wine-gitlab - winehq.org

[PATCH v2 0/7] MR6041: Cmd engine rewrite (part XIX)
by eric pouech (＠epo) 10 Jul '24

10 Jul '24

This is part XIX of cmd engine rewrite. It's the last MR (for now) for setting success/failure return code on internal commands. The remaining todo in this area are due to: - incompletly implemented builtin (eg: TYPE not handling multiple files, not wildcards; CHOICE missing a bunch of features; ...), - some commands (like START) being implemented as external executables, hence forcing RETURN CODE = ERRORLEVEL which is not always wanted, -- v2: cmd: Cleanup transition bits. cmd: Improve return code/errorlevel support for external commands. cmd: Move empty batch command handling to WCMD_batch() callers. cmd: Set success/failure return code for START command. cmd: Set success/failure return code for MKLINK command. cmd: Add more tests for return codes in builtin commands. cmd: Get rid of CTTY command. https://gitlab.winehq.org/wine/wine/-/merge_requests/6041

4 9

Re: [PATCH v3 0/1] MR6037: server: Use a shared_object_t for the dummy object. - approved
by Alexandre Julliard (＠julliard) 10 Jul '24

10 Jul '24

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

1 0

Re: [PATCH v3 0/8] MR6039: jscript: Add support for host functions. - approved
by Alexandre Julliard (＠julliard) 10 Jul '24

10 Jul '24

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

1 0

Re: [PATCH v3 0/2] MR5044: ntdll: Fix some threadpool races. - approved
by Alexandre Julliard (＠julliard) 10 Jul '24

10 Jul '24

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

1 0

Re: [PATCH v5 0/4] MR5237: Use futexes on macOS for thread-ID alerts. - approved
by Alexandre Julliard (＠julliard) 10 Jul '24

10 Jul '24

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

1 0

Re: [PATCH v2 0/1] MR6038: odbc32: Implement SQLSetEnvAttr(SQL_ATTR_ODBC_VERSION). - approved
by Alexandre Julliard (＠julliard) 10 Jul '24

10 Jul '24

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

1 0

Re: [PATCH v4 0/5] MR6023: winegstreamer/wg_transform: Support creating encoder transform. - approved
by Alexandre Julliard (＠julliard) 10 Jul '24

10 Jul '24

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

1 0

Re: [PATCH v2 0/1] MR6035: winhttp: Implement WinHttpQueryOption(WINHTTP_OPTION_URL). - approved
by Alexandre Julliard (＠julliard) 10 Jul '24

10 Jul '24

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

1 0

Re: [PATCH v2 0/1] MR6034: include: Fix typo with XINPUT_DEVSUBTYPE_FLIGHT_STICK name in xinput.h. - approved
by Alexandre Julliard (＠julliard) 10 Jul '24

10 Jul '24

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

1 0

Re: [PATCH v7 0/5] MR5559: include: Fix another 8 typoed GUIDs - approved
by Alexandre Julliard (＠julliard) 10 Jul '24

10 Jul '24

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

1 0

Re: [PATCH v2 0/5] MR6033: include: Fix another 5 typoed GUIDs - approved
by Alexandre Julliard (＠julliard) 10 Jul '24

10 Jul '24

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

1 0

[PATCH v4 0/8] MR6031: Draft: wine: Use atomic locks insted pthread_mutex.
by Grigory Vasilyev (＠h0tc0d3) 10 Jul '24

10 Jul '24

I'm not sure about the need for this solution, so it's a **DRAFT**. For me it is an academic interest to check whether atomic locks will give advantages over pthread_mutex in games. First impressions are that games have become smoother, but need to think about tests that can actually be measured. Before build need define `WINE_USE_ATOMIC_LOCKS`. ```bash export CFLAGS="${CFLAGS} -DWINE_USE_ATOMIC_LOCKS" ``` -- v4: ws2_32: Add atomic lock support. wine32u: Add atomic lock support. winevulkan: Add atomic lock support. ntdll: Add atomic lock support. winewayland: Add atomic lock support. include: Define custom mutex macroses. https://gitlab.winehq.org/wine/wine/-/merge_requests/6031

5 14

Re: [PATCH v11 0/2] MR5969: kernelbase: Ensure null termination in RegGetValue[AW].
by Alex Henrie (＠alexhenrie) 10 Jul '24

10 Jul '24

On Wed Jul 10 19:12:40 2024 +0000, Fabian Maurer wrote: > Is there a specific reason those two functions are separate instead of > RegGetValueA calling RegGetValueW? I don't know. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5969#note_75808

1 0

[PATCH v9 0/1] MR5968: cmd: Do not set enviroment variable when no input is provided by set /p command.
by Vijay Kiran Kamuju (＠infyquest) 10 Jul '24

10 Jul '24

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55662 Tests are not provided as its interactive. -- v9: cmd: Do not set enviroment variable when no input is provided by set /p command. https://gitlab.winehq.org/wine/wine/-/merge_requests/5968

2 1

[PATCH v3 0/8] MR6031: Draft: wine: Use atomic locks insted pthread_mutex.
by Grigory Vasilyev (＠h0tc0d3) 10 Jul '24

10 Jul '24

I'm not sure about the need for this solution, so it's a **DRAFT**. For me it is an academic interest to check whether atomic locks will give advantages over pthread_mutex in games. First impressions are that games have become smoother, but need to think about tests that can actually be measured. Before build need define `WINE_USE_ATOMIC_LOCKS`. ```bash export CFLAGS="${CFLAGS} -DWINE_USE_ATOMIC_LOCKS" ``` -- v3: ws2_32: Add atomic lock support. wine32u: Add atomic lock support. winevulkan: Add atomic lock support. ntdll: Add atomic lock support. winewayland: Add atomic lock support. include: Define custom mutex macroses. https://gitlab.winehq.org/wine/wine/-/merge_requests/6031

4 12

[PATCH v3 0/5] MR6023: winegstreamer/wg_transform: Support creating encoder transform.
by Ziqing Hui (＠zhui) 10 Jul '24

10 Jul '24

-- v3: winegstreamer/wg_transform: Support creating encoder transform. winegstreamer/wg_transform: Introduce transform_create_converter_elements. winegstreamer/wg_transform: Introduce transform_create_decoder_elements. https://gitlab.winehq.org/wine/wine/-/merge_requests/6023

5 9

[PATCH 0/4] MR6045: server: Reimplement mailslots using server-side I/O.
by Elizabeth Figura (＠zfigura) 10 Jul '24

10 Jul '24

This fixes various corner cases. This is not motivated by any particular application. However, the server-side code seems at least as simple as the existing client-side code, is more accurate, and removes a potential source of complication from any future work involving asyncs. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6045

3 5

[PATCH v2 0/2] MR5044: ntdll: Fix some threadpool races.
by Paul Gofman (＠gofman) 10 Jul '24

10 Jul '24

That seems to fix some (difficult to reproduce) crashes in EA Desktop / CEF, usually on shutdown but sometimes during starting up. Currently TpSetWait can set (or clear) the event, while waitqueue_thread_proc() gets woken from NtWaitForMultipleObjects by previously set wait object and call the callback as if new set (or cleared) wait object is signaled. The crashes I was reproducing were always happening when RtlDeregisterWaitEx call is racing with waking the wait and calling the callback. -- v2: ntdll: Make sure wakeups from already unset events are ignored in waitqueue_thread_proc(). https://gitlab.winehq.org/wine/wine/-/merge_requests/5044

3 5

[PATCH v4 0/4] MR5237: Use futexes on macOS for thread-ID alerts.
by Marc-Aurel Zent (＠mzent) 10 Jul '24

10 Jul '24

This also bumps the minimum supported version of macOS to 10.12 to unconditionally assume the ulock syscalls are present. As discussed with @Gcenx this shouldn't be an issue, as the current minimum support version of 10.8 does not work anyways at the moment with only 10.10+ being useable and even then virtually no-one using wine on versions < 10.12 (and also being unsupported by the prebuilt binaries). Bumping to 10.12 would also have the side-benefit of allowing some cleanup of deprecated functions or legacy codepaths (like `mach_continuous_time` vs `mach_absolute_time` and probably much more). In my testing ulock futexes are about 2x faster for synchronization than kevents. Using the syscall wrappers here should provide protection against changing syscall numbers, and given that ulock is also directly used by libc++, the probability of an incompatible API change here is basically zero imo. By default ulock is also process-private, similar to the current linux implementation, cross-process synchronization would require darwin 19+ (macOS 10.15+) with `UL_COMPARE_AND_WAIT_SHARED`. The diff is a bit bigger than it actually is in effect, since https://gitlab.winehq.org/mzent/wine/-/commit/28a8a991c165f0d37dbf78ca3cdee… is only moving some code around and not actually changing anything. -- v4: ntdll: Implement futex_wait() and futex_wake_one() on macOS. ntdll: Simplify futex interface from futex_wake() to futex_wake_one(). ntdll: Use USE_FUTEX to indicate futex support. ntdll: Prefer futex for thread-ID alerts over kqueue. https://gitlab.winehq.org/wine/wine/-/merge_requests/5237

3 6

[PATCH 0/7] MR6041: Cmd engine rewrite (part XIX)
by eric pouech (＠epo) 10 Jul '24

10 Jul '24

This is part XIX of cmd engine rewrite. It's the last MR (for now) for setting success/failure return code on internal commands. The remaining todo in this area are due to: - incompletly implemented builtin (eg: TYPE not handling multiple files, not wildcards; CHOICE missing a bunch of features; ...), - some commands (like START) being implemented as external executables, hence forcing RETURN CODE = ERRORLEVEL which is not always wanted, -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6041

3 9

[PATCH v2 0/8] MR6039: jscript: Add support for host functions.
by Jacek Caban (＠jacek) 10 Jul '24

10 Jul '24

And use new bindings for more MSHTML objects. -- v2: mshtml: Use host object script bindings for Performance class. mshtml: Use host object script bindings for PerformanceTiming class. mshtml: Use host object script bindings for PerformanceNavigation class. mshtml: Use host object script bindings for History class. mshtml: Use host object script bindings for DOMImplementation class. mshtml/tests: Make todo_wine explicit in builtin_toString tests. jscript: Add support for host functions. https://gitlab.winehq.org/wine/wine/-/merge_requests/6039

3 9

[PATCH v4 0/3] MR6004: comctl32: A NULL string fix.
by Aida Jonikienė (＠DodoGTA) 10 Jul '24

10 Jul '24

-- v4: comctl32/tests: Add tests for NULL strings with edit control WM_GETTEXT message. user32: Correctly handle NULL string in WM_GETTEXT AtoW wrapper. https://gitlab.winehq.org/wine/wine/-/merge_requests/6004

3 11

Re: [PATCH v5 0/3] MR5585: mmdevapi: Implement AudioClockAdjustment_SetSampleRate.
by Arek Hiler (＠ivyl) 10 Jul '24

10 Jul '24

On Mon Jul 8 21:43:03 2024 +0000, Etaash Mathamsetty wrote: > cracked the mystery, im pretty sure AUDCLNT_STREAMFLAGS_RATEADJUST is > needed for GetService to work :) This explains things, thanks! I'll amend the tests and add support in `GetService()`. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5585#note_75784

1 0

Re: [PATCH v5 0/3] MR5585: mmdevapi: Implement AudioClockAdjustment_SetSampleRate.
by Arek Hiler (＠ivyl) 10 Jul '24

10 Jul '24

On Tue Jul 9 19:55:30 2024 +0000, Davide Beatrici wrote: > Since the operations are more or less the same as in > `pulse_create_stream()`, could you create a helper function in a > separate commit (that will be first in the list), please? I've been looking into this when originally implementing the function. I had another look now. The only shared parts that I think we could extract are ```c new_bufsize_frames = ceil((stream->duration / 10000000.) * new_ss.rate); size = new_bufsize_frames * 2 * pa_frame_size(&stream->ss); if (NtAllocateVirtualMemory(GetCurrentProcess(), (void **)&new_buffer, zero_bits, &size, MEM_COMMIT, PAGE_READWRITE)) hr = E_OUTOFMEMORY; ``` but I couldn't make it not overly awkward, mostly because of when the variables have to be set and the failure paths / cleanup. I can do two helpers, one for `bufsize_frames` and `size` each unless you have a more detailed suggestion. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5585#note_75783

1 0

[PATCH v2 0/8] MR6031: Draft: wine: Use atomic locks insted pthread_mutex.
by Grigory Vasilyev (＠h0tc0d3) 10 Jul '24

10 Jul '24

I'm not sure about the need for this solution, so it's a **DRAFT**. For me it is an academic interest to check whether atomic locks will give advantages over pthread_mutex in games. First impressions are that games have become smoother, but need to think about tests that can actually be measured. Before build need define `WINE_USE_ATOMIC_LOCKS`. ```bash export CFLAGS="${CFLAGS} -DWINE_USE_ATOMIC_LOCKS" ``` -- v2: ws2_32: Add atomic lock support. wine32u: Add atomic lock support. winevulkan: Add atomic lock support. ntdll: Add atomic lock support. winewayland: Add atomic lock support. include: Define custom mutex macroses. msxml3: Fix compilation errors with Clang 18. configure: Change C standard to C17. https://gitlab.winehq.org/wine/wine/-/merge_requests/6031

4 15

2025

2024

2023

2022

wine-gitlab