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

[PATCH 0/1] MR7132: gdiplus/tests: Trace locked bitmap data on failure.
by Esme Povirk (＠madewokherd) 11 Jan '25

11 Jan '25

For debugging test failures like this one: https://test.winehq.org/data/8331e7db0d540656a047f415184a30df5acbc3f9/linux… -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7132

2 1

[PATCH v2 0/1] MR7055: appwiz.cpl: Account for unicode characters in XDG_CACHE_HOME.
by Esme Povirk (＠madewokherd) 11 Jan '25

11 Jan '25

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57338 -- v2: appwiz.cpl: Account for unicode characters in XDG_CACHE_HOME. https://gitlab.winehq.org/wine/wine/-/merge_requests/7055

2 2

Re: [PATCH v4 0/1] MR7091: windowscodecs: Make values returned from CanConvert consistent - approved
by Esme Povirk (＠madewokherd) 11 Jan '25

11 Jan '25

This merge request was approved by Esme Povirk. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7091

1 0

Re: [PATCH v2 0/0] MR7130: (INVALID) - closed
by Jinoh Kang (＠iamahuman) 11 Jan '25

11 Jan '25

This merge request was closed by Jinoh Kang. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7130

1 0

Re: [PATCH v2 0/7] MR6911: Draft: ntdll: Add WaitCompletionPacket.
by Jinoh Kang (＠iamahuman) 11 Jan '25

11 Jan '25

Jinoh Kang (@iamahuman) commented about server/thread.c: > + { > + struct wait_completion_packet *packet; > + > + LIST_FOR_EACH_ENTRY( packet, &obj->wait_completion_packet_queue, struct wait_completion_packet, entry ) > + { > + list_remove( &packet->entry ); > + release_object( packet->target ); > + packet->in_object_packet_queue = 0; > + packet->target = NULL; > + > + add_completion( packet->completion, packet->ckey, packet->cvalue, packet->status, > + packet->information, packet ); > + packet->in_completion_queue = 1; > + } > + } > + General advice: you should try not to leak implementation details of certain object (wait completion packet) into other types of objects. For example, if this piece of code is still to be preserved, you should move this into a helper completion.c, declare as extern in object.h (or file.h), and call that. That way, you don't have to define the entire `struct wait_completion_packet` in a shared header file. If this were factored into a helper, a simple forward declaration would have sufficed in the `file.h` side. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6911#note_91785

1 0

[PATCH 0/1] MR7129: ntoskrnl.exe: Fix IRQL mismatch between cancel spin lock acquire and release.
by Jinoh Kang (＠iamahuman) 11 Jan '25

11 Jan '25

Use the irql assigned by IoAcquireCancelSpinLock, not the (yet-to-be-assigned) irp->CancelIrql. This is for readability. No semantic change is expected with unmodified ntoskrnl.exe/hal.dll. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7129

3 2

[PATCH v2 0/1] MR7129: ntoskrnl.exe: Fix IRQL mismatch between cancel spin lock acquire and release.
by Jinoh Kang (＠iamahuman) 11 Jan '25

11 Jan '25

Use the irql assigned by IoAcquireCancelSpinLock, not the (yet-to-be-assigned) irp->CancelIrql. This is for readability. Since IoReleaseCancelSpinLock() ignores the IRQL argument anyway, no semantic change is expected with unmodified ntoskrnl.exe/hal.dll. -- v2: ntoskrnl.exe: Fix IRQL mismatch between cancel spin lock acquire and release. https://gitlab.winehq.org/wine/wine/-/merge_requests/7129

2 1

Re: [PATCH v2 0/1] MR7124: mfmediaengine: Fix maths in scaling check. - approved
by Alexandre Julliard (＠julliard) 10 Jan '25

10 Jan '25

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

1 0

Re: [PATCH v2 0/4] MR7057: Draft: amstream: Stop stream while running CreateSample. Feedback needed. :-)
by Santino Mazza (＠tati) 10 Jan '25

10 Jan '25

On Sat Dec 21 08:41:22 2024 +0000, Anton Baskanov wrote: > The correct way to fix this is to implement a custom allocator like it > is done in the native AMDirectDrawStream. This will allow changing the > format on the fly without stopping the stream. But it of course requires > much more work, including adding format change support to decoders. Thanks for the feedback! I started reading about dynamic format changing and dynamic reconnecting from your comment, so now I have a better idea on how to continue with this. :-) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7057#note_91771

1 0

[PATCH v3 0/1] MR7108: tools: Add manual-run job for new-style wow64.
by Jinoh Kang (＠iamahuman) 10 Jan '25

10 Jan '25

The new job is only triggered when a developer manually starts it. Before run, it initially appears as a "skipped" job when the pipeline starts. -- v3: tools: Add manual-run job for new-style wow64. https://gitlab.winehq.org/wine/wine/-/merge_requests/7108

3 2