- Wine-GitLab - list.winehq.org

Re: [PATCH v2 0/4] MR413: ntoskrnl: Enumerate child devices on a separate thread. - approved
by Alexandre Julliard (＠julliard) July 11, 2022

July 11, 2022

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

1 0

Re: [PATCH v10 0/1] MR320: riched20/tests: Test for ITextDocument::Undo and ITextDocument::Redo (#53289). - approved
by Alexandre Julliard (＠julliard) July 11, 2022

July 11, 2022

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

1 0

Re: [PATCH v2 0/3] MR415: ddraw/tests: Build without -DWINE_NO_LONG_TYPES. - approved
by Alexandre Julliard (＠julliard) July 11, 2022

July 11, 2022

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

1 0

[PATCH v3 0/1] MR423: ieframe fix
by David Kahurani (＠kahurani) July 11, 2022

July 11, 2022

Signed-off-by: David Kahurani <k.kahurani(a)gmail.com> -- v3: ieframe: Fix memory leak in open_dde_url https://gitlab.winehq.org/wine/wine/-/merge_requests/423

3 2

[PATCH 0/1] MR424: ntdll: Add ".globl" for asm labels referenced from C.
by Brendan Shanks (＠bshanks) July 11, 2022

July 11, 2022

Fixes syscall faults on 32-bit macOS. Spotted by Stefan D��singer. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/424

2 1

[PATCH v2 0/1] MR423: ieframe fix
by David Kahurani (＠kahurani) July 11, 2022

July 11, 2022

Signed-off-by: David Kahurani <k.kahurani(a)gmail.com> -- v2: ieframe: Fix memory leak and possible reference leak https://gitlab.winehq.org/wine/wine/-/merge_requests/423

4 4

[PATCH 0/1] MR423: ieframe fix
by David Kahurani (＠kahurani) July 11, 2022

July 11, 2022

Signed-off-by: David Kahurani <k.kahurani(a)gmail.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/423

3 2

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Jinoh Kang (＠iamahuman) July 11, 2022

July 11, 2022

Jinoh Kang (@iamahuman) commented about dlls/kernelbase/file.c: > */ > BOOL WINAPI DECLSPEC_HOTPATCH CancelSynchronousIo( HANDLE thread ) > { > - FIXME( "(%p): stub\n", thread ); > - SetLastError( ERROR_CALL_NOT_IMPLEMENTED ); > - return FALSE; > + IO_STATUS_BLOCK io; > + > + return set_ntstatus( NtCancelSynchronousIoFile( thread, NULL, &io ) ); Per Wine code style, you should omit the second space in `&io ) )`. ```suggestion:-0+0 return set_ntstatus( NtCancelSynchronousIoFile( thread, NULL, &io )); ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_3876

1 0

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Jinoh Kang (＠iamahuman) July 11, 2022

July 11, 2022

Jinoh Kang (@iamahuman) commented about dlls/kernel32/tests/pipe.c: > + > + SetLastError(0xdeadbeef); > + res = pCancelSynchronousIo(GetCurrentThread()); > + ok(!res, "CancelSynchronousIo succeeded unexpectedly\n"); > + todo_wine > + ok(GetLastError() == ERROR_NOT_FOUND, > + "In CancelSynchronousIo failure, expected ERROR_NOT_FOUND, got %ld\n", GetLastError()); > + > + /* synchronous i/o */ > + pipe = CreateNamedPipeA(PIPENAME, PIPE_ACCESS_DUPLEX, > + PIPE_TYPE_MESSAGE | PIPE_READMODE_MESSAGE | PIPE_WAIT, > + 1, 1024, 1024, NMPWAIT_USE_DEFAULT_WAIT, NULL); > + ok(pipe != INVALID_HANDLE_VALUE, "CreateNamedPipe failed with %lu\n", GetLastError()); > + thread = CreateThread(NULL, 0, synchronousIoThreadMain, pipe, 0, NULL); > + /* wait for thread to start listening */ > + Sleep(100); How about the following race-free approach? ```suggestion:-0+0 /* wait for I/O to start, which transitions the pipe handle from signaled to nonsignaled state. */ while ((wait_status = WaitForSingleObject(pipe, 0)) == WAIT_OBJECT_0) Sleep(1); ok(wait_status == WAIT_TIMEOUT, "WaitForSingleObject returned %lu (error %lu)\n", wait_status, GetLastError()); ``` Ditto for other places. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_3874

1 0

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Jinoh Kang (＠iamahuman) July 11, 2022

July 11, 2022

Jinoh Kang (@iamahuman) commented about server/async.c: > return woken; > } > > +static int cancel_blocking( struct process *process, struct thread *thread, client_ptr_t iosb ) > +{ > + struct async *async; > + int woken = 0; > + > +restart: > + LIST_FOR_EACH_ENTRY( async, &process->asyncs, struct async, process_entry ) > + { > + if (async->terminated || async->canceled) continue; > + if (async->blocking && (async->thread == thread) && Maybe you forgot to remove the extraneous parentheses? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_3875

1 0

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Jinoh Kang (＠iamahuman) July 11, 2022

July 11, 2022

> the test would hang on wine before the implementation is in place. Thanks! I missed that the current test cannot verify the Windows behaviour before the commit that introduces the Wine implementation. > the CloseHandle before the WaitForSingleObject is a workaround to get the thread to exit I'm afraid that closing a handle in use is racy by nature; it's effectively pulling the rug out from underneath. A better approach would be to use `CancelIoEx` on the `OVERLAPPED` structure, or `NtCancelIoFileEx` on the `IO_STATUS_BLOCK` structure to cancel the pending operation. > > moving the CloseHandle was meant to show that workaround is no longer needed and that it now performs like windows > > i can remove that if preferred -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_3873

1 0

[PATCH v3 0/2] MR360: Draft: winepulse.drv: add support for containerId property from sysfs path
by Claire (＠ClearlyClaire) July 11, 2022

July 11, 2022

Some games with support for the haptic feedback and speaker features of the Sony DualSense controller select the controller's audio output by filtering on the ContainerId IMMDevice property to find one that matches the controller's HID's. This MR, together with !359, adds support for exposing such a ContainerId to applications. I marked this MR as a draft because I understand that the way the ��guid�� is generated is far from ideal. Furthermore, I will need to map from sysfs to container Id in other components as well (`winebus` for the HID device, and possibly `winealsa` and other audio drivers), so moving that part elsewhere would make sense. However, I think I will need help with those tasks. -- v3: winepulse: Add support for containerId property from sysfs path winepulse: Store PulseAudio device's sysfs path when available https://gitlab.winehq.org/wine/wine/-/merge_requests/360

2 2

[PATCH v4 0/4] MR359: mmdevapi: copy ContainerID property from audio driver if available
by Claire (＠ClearlyClaire) July 11, 2022

July 11, 2022

Some games with support for the haptic feedback and speaker features of the Sony DualSense controller select the controller's audio output by filtering on the ContainerId IMMDevice property to find one that matches the controller's HID's. This MR allows this information to be accessible from the IMMDevice's property store when the audio driver provides it (none for now, but I intend to add that feature to `winepulse.drv` and maybe `winealsa.drv`) This is made specific to containerId rather than supporting VT_CLSID on every property because Wine currently stores VT_BLOBs directly in the registry value, which does not allow us to safely disambiguate between VT_CLSID and VT_BLOB values when reading from registry. -- v4: mmdevapi: Invalidate ContainerID of unavailable audio devices mmdevapi: copy ContainerID from audio driver if available mmdevapi: decode ContainerId property to CLSID in MMDevice_GetPropValue mmdevapi: support VT_CLSID for containerId property in MMDevice_SetPropValue https://gitlab.winehq.org/wine/wine/-/merge_requests/359

3 4

[PATCH 0/6] MR420: imm32: win32u conversion
by Jacek Caban (＠jacek) July 11, 2022

July 11, 2022

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

3 7

[PATCH 0/1] MR419: user32/tests: Check that a message window brought topmost does not defocus other windows.
by Giovanni Mascellani (＠giomasce) July 11, 2022

July 11, 2022

libOVR brings a message window topmost claiming that this way it receives WM_DEVICECHANGE faster. Currently, in Wine, this causes a WM_KILLFOCUS to be sent to the actual topmost window, which in turn causes "Shantae: Risky's Revenge" to minimize. The effect is that the game automatically minimizes as soon as it is launched, which is incorrect. Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/419

2 1

[PATCH v2 0/1] MR409: crypt32: Recognize 'Microsoft Root Certificate Authority 2011' when verifying...
by Paul Gofman (＠gofman) July 11, 2022

July 11, 2022

crypt32: Recognize 'Microsoft Root Certificate Authority 2011' when verifying the Microsoft root policy. Signed-off-by: Paul Gofman <pgofman(a)codeweavers.com> -- v2: crypt32: Support MICROSOFT_ROOT_CERT_CHAIN_POLICY_CHECK_APPLICATION_ROOT_FLAG. https://gitlab.winehq.org/wine/wine/-/merge_requests/409

3 3

[PATCH v4 0/27] MR398: Draft: xactengine3_7/tests: Test notifications when loading a wave bank.
by Giovanni Mascellani (＠giomasce) July 11, 2022

July 11, 2022

"The King of Fighters '98 Ultimate Match Final Edition" depends on this behavior. At least, the build I have is; it seems other builds are not. Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> -- v4: xactengine3_7/tests: Test notifications when loading a wave bank. fixup! include/wine: Add ANSI color codes. ntdll: Dump executable. include/wine: Add ANSI color codes. ntdll: Introduce x86emu. ntdll: Allowing injecting an int3. ntdll: Allow setting a watchpoint. include/wine: Introduce STAR to dereference a generic number. widl: Generate empty interface implementations. ntdll: Log architecture. ntdll: Implement debug writing to file. ntdll: Mark Wine code in mmap output. ntdll: Support dynamically enabling trace messages. ntdll: Log signals early. ntdll: Dump a single symbol. ntdll: Capture stack backtrace from UNIX side. ntdll: Add facility to disassemble code. ntdll: Print the command line when a process is started. ntdll: Dump memory mappings and unmappings. ntdll: Make dynamic unwind entries accessible from UNIX side. ntdll: Add +microsecs channel for precise timestamps. ntdll: Log return address. ntdll: Log file and line number. ntdll: Do not suppress useful log fields on \1. ntdll: Implement log locking and disabling. ntdll: Add a mechanism to ease development calls. ntdll: Support gio channel by default. This merge request has too many patches to be relayed via email. Please visit the URL below to see the contents of the merge request. https://gitlab.winehq.org/wine/wine/-/merge_requests/398

2 2

[PATCH v3 0/27] MR398: Draft: xactengine3_7/tests: Test notifications when loading a wave bank.
by Giovanni Mascellani (＠giomasce) July 11, 2022

July 11, 2022

"The King of Fighters '98 Ultimate Match Final Edition" depends on this behavior. At least, the build I have is; it seems other builds are not. Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> -- v3: xactengine3_7/tests: Test notifications when loading a wave bank. fixup! include/wine: Add ANSI color codes. ntdll: Dump executable. include/wine: Add ANSI color codes. ntdll: Introduce x86emu. ntdll: Allowing injecting an int3. ntdll: Allow setting a watchpoint. include/wine: Introduce STAR to dereference a generic number. widl: Generate empty interface implementations. ntdll: Log architecture. ntdll: Implement debug writing to file. ntdll: Mark Wine code in mmap output. ntdll: Support dynamically enabling trace messages. ntdll: Log signals early. ntdll: Dump a single symbol. ntdll: Capture stack backtrace from UNIX side. ntdll: Add facility to disassemble code. ntdll: Print the command line when a process is started. ntdll: Dump memory mappings and unmappings. ntdll: Make dynamic unwind entries accessible from UNIX side. ntdll: Add +microsecs channel for precise timestamps. ntdll: Log return address. ntdll: Log file and line number. ntdll: Do not suppress useful log fields on \1. ntdll: Implement log locking and disabling. ntdll: Add a mechanism to ease development calls. ntdll: Support gio channel by default. This merge request has too many patches to be relayed via email. Please visit the URL below to see the contents of the merge request. https://gitlab.winehq.org/wine/wine/-/merge_requests/398

1 1

[PATCH 0/2] MR418: mfplat: Support and test YV12, I410 and IYUV image formats.
by Giovanni Mascellani (＠giomasce) July 11, 2022

July 11, 2022

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

2 2

[PATCH 0/3] MR417: riched20: Add font fallback support.
by Jactry Zeng (＠jactry) July 11, 2022

July 11, 2022

Signed-off-by: Jactry Zeng <jzeng(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/417

2 3

[PATCH 0/3] MR416: Cladun X2 crash fixes
by Zhiyi Zhang (＠zhiyi) July 11, 2022

July 11, 2022

All these patches are required to fix Cladun X2 crash at start. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/416

2 3

[PATCH v2 0/4] MR406: win32u: Introduce new add_modes device manager callback.
by Rémi Bernon July 11, 2022

July 11, 2022

-- v2: win32u: Read and cache adapter modes from the registry. win32u: Introduce new add_mode device manager callback. winemac.drv: Introduce new display_mode_to_devmode helper. winex11.drv: Set desktop settings handler before updating display devices. https://gitlab.winehq.org/wine/wine/-/merge_requests/406

2 6

[PATCH v9 0/1] MR320: riched20/tests: Test for ITextDocument::Undo and ITextDocument::Redo (#53289).
by Jinoh Kang (＠iamahuman) July 11, 2022

July 11, 2022

Signed-off-by: Jinoh Kang <jinoh.kang.kr(a)gmail.com> -- v9: riched20/tests: Test for ITextDocument::Undo and ITextDocument::Redo. https://gitlab.winehq.org/wine/wine/-/merge_requests/320

3 2

[PATCH 0/3] MR415: ddraw/tests: Build without -DWINE_NO_LONG_TYPES.
by Zebediah Figura (＠zfigura) July 11, 2022

July 11, 2022

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

2 3

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Daniel Lehman (＠dlehman25) July 11, 2022

July 11, 2022

the test would hang on wine before the implementation is in place. the CloseHandle before the WaitForSingleObject is a workaround to get the thread to exit moving the CloseHandle was meant to show that workaround is no longer needed and that it now performs like windows i can remove that if preferred -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_3840

1 0