July 2023 - Wine-GitLab - list.winehq.org

[PATCH 0/4] MR3442: CreateProcess fixes for process group.
by eric pouech (＠epo) July 28, 2023

July 28, 2023

This serie: - adds a couple of tests to CreateProcess to cover ctrl-c blocking inheritance flag and process group creation - populates RTL_USER_PROCESS_PARAMETERS.ProcessGroupId - separates new process group creation from ctrl-c inheritance flag -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3442

3 5

Re: [PATCH v5 0/5] MR283: vkd3d-shader/dxil: Implement DX instruction StoreOutput.
by Giovanni Mascellani (＠giomasce) July 28, 2023

July 28, 2023

Giovanni Mascellani (@giomasce) commented about libs/vkd3d-shader/dxil.c: > FIXME("Unhandled dxil instruction %u.\n", record->code); > return VKD3D_ERROR_INVALID_SHADER; > } > + /* Allocation failure. */ That's not necessarily the case, is it? Any call to `vkd3d_shader_parser_error()`, such as in `dxil_record_validate_operand_min_count()`, could trigger this failure. More in general, I think we there is some confusion between whether to use the parser `failed` bit vs using the return value. We could use the same convention as the HLSL compiler: `failed` is set as soon as an error is found, and it should always set via `vkd3d_shader_parser_error()` so that a diagnostic is always available; an error return value should only be used when an error was found and parsing must halt immediately. In this scheme you should never look at `failed` to decide what to return, because you don't know if `failed` was set because of an error that still allows parsing or not. You have to genuinely propagate the return code at each level. It is useful to have memory allocation functions that also set the `failed` bit, like the HLSL compiler. -- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/283#note_40646

1 0

Re: [PATCH v5 0/5] MR283: vkd3d-shader/dxil: Implement DX instruction StoreOutput.
by Giovanni Mascellani (＠giomasce) July 28, 2023

July 28, 2023

Giovanni Mascellani (@giomasce) commented about libs/vkd3d-shader/dxil.c: > ++block_idx; > code_block = (block_idx < function->block_count) ? function->blocks[block_idx] : NULL; This was already here before, but I just realized that setting `code_block` to `NULL` might give a few problems if there are some other instructions (which shouldn't be there, but the shader might be untrusted input). Specifically, various lines in this loop assume that `code_block` is valid, and if it's not an `assert()` could be triggered. -- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/283#note_40645

1 0

[PATCH 0/1] MR3441: uiautomationcore: Fix a sizeof() argument (Coverity).
by Sven Baars (＠sbaars) July 28, 2023

July 28, 2023

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

3 2

[PATCH 0/1] MR3440: winevulkan: Update to VK spec version 1.3.260.
by Georg Lehmann (＠DadSchoorse) July 28, 2023

July 28, 2023

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

2 1

[PATCH 0/6] MR3439: mf: Implement basic urlmon scheme handler.
by Rémi Bernon July 28, 2023

July 28, 2023

Let The Good Life able to play its intro video, the game creates a source reader from a `http://localhost:6000/<random-hash>` URL. This should also probably work with other games playing streams over http(s). This is a very basic implementation, using urlmon, and it will download the entire stream to a local temporary file before playback. An more optimized implementation would probably use WinHttp and range requests to partially download the requested stream segments, but this can be implemented in a future change. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3439

2 7

[PATCH v23 0/2] MR3391: mshtml: implement window.MutationObserver with MutationObserver stub
by Yuxuan Shui (＠yshui) July 28, 2023

July 28, 2023

This get us pass the "Update your browser" blocker in adobe's sign-in page. The page itself doesn't make use of `window.MutationObserver`. However the sign-in page is still broken. -- v23: mshtml: add stubs for MutationObserver methods mshtml: implement window.MutationObserver with MutationObserver stub https://gitlab.winehq.org/wine/wine/-/merge_requests/3391

4 6

[PATCH 0/1] MR3438: klist: Match native output for encryption type.
by Hans Leidekker (＠hans) July 28, 2023

July 28, 2023

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

2 1

Re: [PATCH v3 0/1] MR3389: Use PCSC framework instead of pcsclite on macOS
by Hans Leidekker (＠hans) July 28, 2023

July 28, 2023

Looks like the build is using cached configure checks. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3389#note_40624

1 0

Re: [PATCH v3 0/1] MR3389: Use PCSC framework instead of pcsclite on macOS
by Huw Davies (＠huw) July 28, 2023

July 28, 2023

It's inconsistent which is strange: some runs are succeeding, others are not. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3389#note_40623

1 0