November 2024 - Wine-GitLab

[PATCH 0/2] MR6921: ntdll: Fix reported exception code for some brk immediates.
by Billy Laws (＠bylaws) Nov. 29, 2024

Nov. 29, 2024

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

4 4

[PATCH v10 0/1] MR6919: bcp47langs: Add tests for GetUserLanguages.
by Vijay Kiran Kamuju (＠infyquest) Nov. 29, 2024

Nov. 29, 2024

-- v10: bcp47langs: Add tests for GetUserLanguages. https://gitlab.winehq.org/wine/wine/-/merge_requests/6919

3 2

[PATCH v3 0/2] MR6838: msi: Dynamically determine supported package architectures.
by Billy Laws (＠bylaws) Nov. 29, 2024

Nov. 29, 2024

I wish there was a nicer way to do this, but SystemSupportedProcessorArchitectures is incomplete on ARM64EC even on Windows so there isn't really a cleaner way I can find. -- v3: msi: Dynamically determine supported package architectures. msi: Assume PLATFORM_INTEL if the template property is missing. https://gitlab.winehq.org/wine/wine/-/merge_requests/6838

5 5

[PATCH v2 0/1] MR6894: d3dx9: Remove superfluous null check (Coverity)
by Fabian Maurer (＠DarkShadow44) Nov. 29, 2024

Nov. 29, 2024

-- v2: d3dx9: Remove superfluous null check (Coverity) https://gitlab.winehq.org/wine/wine/-/merge_requests/6894

3 3

[PATCH v9 0/1] MR6919: Draft: bcp47langs: Add tests for GetUserLanguages.
by Vijay Kiran Kamuju (＠infyquest) Nov. 29, 2024

Nov. 29, 2024

-- v9: bcp47langs: Add tests for GetUserLanguages. https://gitlab.winehq.org/wine/wine/-/merge_requests/6919

3 2

[PATCH v5 0/8] MR6800: mf: Handle unexpected media source shutdown in a media session.
by Conor McCarthy (＠cmccarthy) Nov. 29, 2024

Nov. 29, 2024

-- v5: winegstreamer: Send media source shutdown notification via IMFMediaShutdownNotify. mf: Introduce IMFMediaShutdownNotify for notification of media source shutdown. https://gitlab.winehq.org/wine/wine/-/merge_requests/6800

3 9

[PATCH v4 0/8] MR6800: winegstreamer: Lock the media source critical section while using the event queue.
by Conor McCarthy (＠cmccarthy) Nov. 29, 2024

Nov. 29, 2024

These changes are the documented way to implement IMFMediaEventGenerator for a media source. On top of !6783 it seems to greatly reduce the chance of hanging while switching characters in Killsquad if tracing is not enabled. -- v4: winegstreamer: Send media source shutdown notification via IMFMediaShutdownNotify. mf: Introduce IMFMediaShutdownNotify for notification of media source shutdown. mf: Handle media source Start() failure due to source shutdown. mf: Handle media source event subscription failure due to source shutdown. mf: Handle media source BeginGetEvent() failure due to shutdown. mf: Handle media source EndGetEvent() failure due to shutdown. mf/tests: Add tests for shutting down a media source used in a session. winegstreamer: Return the result code from media_source_Pause(). https://gitlab.winehq.org/wine/wine/-/merge_requests/6800

3 10

[PATCH v2 0/2] MR6904: ntdll.dll: Update NtQueryDirectoryFile to align with current Windows behaviour, and update tests accordingly
by Eugene McArdle (＠eugenemcardle) Nov. 29, 2024

Nov. 29, 2024

This PR updates the behaviour of `NtQueryDirectoryFile`, bringing it in line with current Windows behaviour. The need for this update was discovered when attempting to build the Unreal Engine with MSVC under Wine. In certain cases conditional include statements do not behave as expected, due to MSVC depending on undocumented behaviour of `NtQueryDirectoryFile`. We ran tests on multiple versions of Windows, and discovered that the behaviour has changed since the original Wine implementation, but the documentation has not. The source code for our test tool, and a set of results can be found [here](https://github.com/TensorWorks/NtQueryDirectoryFile-Test). As of Windows 8, calling `NtQueryDirectoryFile` with a re-used handle, a new mask, and setting the `RestartScan` flag to True, causes the cached results to be erased and a new scan to be performed with the updated mask. Currently, Wine performs as did earlier versions of Windows, where the changed mask is ignored, and the cache is reused. This can cause `NtQueryDirectoryFile` under Wine to falsely report that files exist, when they do not. This PR corrects this behaviour, invalidating the cache when required. Implementing this exposed further undocumented behaviour of `NtQueryDirectoryFile`, where a search for a non-existent file will return either `STATUS_NO_MORE_FILES` or `STATUS_NO_SUCH_FILE`, depending on whether or not the handle had been previously used regardless of the value of `RestartScan`. This was reflected in a `winetest` which allowed for the response to be either `STATUS_SUCCESS` or `STATUS_NO_MORE_FILES`. This test has been updated to only allow current Windows behaviour, and `NtQueryDirectoryFile` will return either `STATUS_NO_MORE_FILES` or `STATUS_NO_SUCH_FILE` as appropriate. This patch also adds unit tests for the new behaviour of `NtQueryDirectoryFile`. These tests pass when running `winetest` under Windows, and under Wine with these changes in place, but they will fail under older versions of Wine. -- v2: ntdll: Test updated NtQueryDirectoryFile behaviour if scan is reset with a new mask ntdll: Update NtQueryDirectoryFile to purge cache if scan is reset with a new mask https://gitlab.winehq.org/wine/wine/-/merge_requests/6904

2 3

Re: [PATCH v8 0/1] MR6425: include: Name parameters in LPPROGRESS_ROUTINE typedef - closed
by Ostap Brehin (＠osbre) Nov. 28, 2024

Nov. 28, 2024

This merge request was closed by Ostap Brehin. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6425

1 0

Re: [PATCH v8 0/1] MR6425: include: Name parameters in LPPROGRESS_ROUTINE typedef
by Nikolay Sivov (＠nsivov) Nov. 28, 2024

Nov. 28, 2024

I suggest we close this, it's nothing but a cosmetic change. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6425#note_89177

1 0