November 2024 - wine-gitlab

Re: [PATCH v2 0/1] MR5792: server: Avoid calling set_event from within msg_queue_add_queue.
by Elizabeth Figura (＠zfigura) 20 Nov '24

20 Nov '24

What was the reason for this change? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5792#note_88204

1 0

[PATCH v4 0/6] MR6850: mshtml: Expose the props properly for Node prototype chain.
by Gabriel Ivăncescu (＠insn) 20 Nov '24

20 Nov '24

This is just nodes for now. The tests in the last patch may seem a bit overkill now, but it will be used for all the other prototypes, so they'll only need extra entries in the array/table, making it way simpler and less redundant. -- v4: mshtml: Don't expose swapNode from NodePrototype. mshtml: Don't expose replaceNode from NodePrototype. mshtml: Don't expose removeNode from NodePrototype. mshtml: Expose ownerDocument from NodePrototype. mshtml/tests: Add initial tests for prototype chain props. mshtml: Enumerate all own builtin props from host object's NextProperty. https://gitlab.winehq.org/wine/wine/-/merge_requests/6850

3 7

[PATCH v3 0/6] MR6850: mshtml: Expose the props properly for Node prototype chain.
by Gabriel Ivăncescu (＠insn) 20 Nov '24

20 Nov '24

This is just nodes for now. The tests in the last patch may seem a bit overkill now, but it will be used for all the other prototypes, so they'll only need extra entries in the array/table, making it way simpler and less redundant. -- v3: mshtml: Don't expose swapNode from NodePrototype. mshtml: Don't expose replaceNode from NodePrototype. mshtml: Don't expose removeNode from NodePrototype. mshtml: Expose ownerDocument from NodePrototype. mshtml/tests: Add initial tests for prototype chain props. mshtml: Enumerate all own builtin props from host object's NextProperty. https://gitlab.winehq.org/wine/wine/-/merge_requests/6850

3 7

[PATCH 0/1] MR6865: jscript: Don't use bool as a variable name.
by Gerald Pfeifer (＠gerald) 20 Nov '24

20 Nov '24

This is a reserved identifier in C23 and would break with GCC 15 moving to -std=gnu23 (which is C23 plus extensions). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6865

4 3

[PATCH v3 0/2] MR4392: server: Don't crash when opening null path with a console handle as RootDirectory.
by Jinoh Kang (＠iamahuman) 20 Nov '24

20 Nov '24

Preparatory patch for revival of https://gitlab.winehq.org/wine/wine/-/merge_requests/498. -- v3: server: Don't crash when opening null path with a console server as RootDirectory. server: Don't crash when opening null path with a console handle as RootDirectory. https://gitlab.winehq.org/wine/wine/-/merge_requests/4392

3 3

[PATCH v2 0/1] MR6864: mshtml: Don't use cycle collection for nsChannel.
by Gabriel Ivăncescu (＠insn) 20 Nov '24

20 Nov '24

Tested and doesn't seem to be leaking anything extra. -- v2: mshtml: Don't use cycle collection for nsChannel. https://gitlab.winehq.org/wine/wine/-/merge_requests/6864

3 2

[PATCH 0/1] MR6864: mshtml: Don't use cycle collection for nsChannel.
by Gabriel Ivăncescu (＠insn) 20 Nov '24

20 Nov '24

Tested and doesn't seem to be leaking anything extra. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6864

1 0

Re: [PATCH v3 0/3] MR5970: win32u: Use the thread message queue shared memory in peek_message.
by sagi trashbin (＠trashbin) 20 Nov '24

20 Nov '24

Does the new throttling mechanism introduce input lag? Testing with a simple SDL2 app polling for events, input delay is 9.8ms averaged over 100 clicks, but native application build is only 1.05ms. I implemented the mechanism used before with 200 message count and the input lag is only 1.2ms, but I suppose in real game there would be a big CPU usage spike -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5970#note_88177

1 0

[PATCH 0/1] MR6859: mshtml: Copy POST data in BindStatusCallback_GetBindInfo().
by Paul Gofman (＠gofman) 20 Nov '24

20 Nov '24

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

5 12

Re: [PATCH v7 0/2] MR6059: mf: Retry PROCESSINPUTNOTIFY if TRANSFORM_TYPE_NOT_SET is returned.
by Nikolay Sivov (＠nsivov) 20 Nov '24

20 Nov '24

Nikolay Sivov (@nsivov) commented about dlls/mf/evr.c: > } > > if (SUCCEEDED(IMFTransform_ProcessInput(stream->parent->mixer, stream->id, sample, 0))) > - IMFVideoPresenter_ProcessMessage(stream->parent->presenter, MFVP_MESSAGE_PROCESSINPUTNOTIFY, 0); > + { > + while (hr == S_OK && (hr = IMFVideoPresenter_ProcessMessage(stream->parent->presenter, MFVP_MESSAGE_PROCESSINPUTNOTIFY, 0)) == MF_E_TRANSFORM_TYPE_NOT_SET) > + hr = IMFVideoPresenter_ProcessMessage(stream->parent->presenter, MFVP_MESSAGE_INVALIDATEMEDIATYPE, 0); > + } If it works like that, sure. But could we turn "hr == S_OK" to a loop break condition, after ProcessMessage. Or simply as "if ((hr == ProcessMessage()) != S_OK) break;. I think that would be more readable and easy to follow. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6059#note_88170

1 0

2025

2024

2023

2022

wine-gitlab November 2024