- wine-gitlab - winehq.org

Re: [PATCH v2 0/1] MR7027: po: Update Japanese translation. - approved
by Alexandre Julliard (＠julliard) 16 Dec '24

16 Dec '24

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

1 0

Re: [PATCH v3 0/2] MR7013: odbc32: Add null pointer checks. - approved
by Alexandre Julliard (＠julliard) 16 Dec '24

16 Dec '24

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

1 0

[PATCH 0/2] MR7014: winex11: Move GL/VK offscreen if the clipping region is NULLREGION.
by Rémi Bernon (＠rbernon) 16 Dec '24

16 Dec '24

NULLREGION means that the entire window should be clipped out, and its children displayed instead. The change broke some offscreening decisions when both parent and children have a GL drawable created but drawing actually happens on the child window. Fixes: 786d9d1685ac220081b10cc779d4d331ddd2fc52 Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57503 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7014

3 3

[PATCH v2 0/2] MR7032: Add embeddedbrowserwebview aka. WebView2 DLL stub.
by Bernhard Kölbl (＠besentv) 16 Dec '24

16 Dec '24

(For after the code freeze.) I think we want to have this DLL living in Wine for easier development, and probably dynamically load our custom Chromium fork from here. (just like MSHTML and wine-Gecko) The code for that fork could then be created in its own repo. -- v2: embeddedbrowserwebview: Create CreateWebViewEnvironmentWithOptionsInternal stub. embeddedbrowserwebview: Add stub dll. https://gitlab.winehq.org/wine/wine/-/merge_requests/7032

5 11

[PATCH 0/2] MR7031: CR/LF in listview items.
by eric pouech (＠epo) 16 Dec '24

16 Dec '24

Some app inserts textual items in listview with CR/LF in them. These are not displayed on Windows, while Wine draws an empty square box for each CR or LF. Manual testing on Windows show that CR or LF are not displayed, while other characters < 32 are displayed. A couple of tests in this MR show that: - the CR / LF are stored in listview (as they can be retrieved) - the CR / LF are somehow accounted in string width So, try to fix this by not passing CR nor LF to DrawText(). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7031

4 4

[PATCH 0/2] MR7032: Add embeddedbrowserwebview aka. WebView2 DLL stub.
by Bernhard Kölbl (＠besentv) 16 Dec '24

16 Dec '24

(For after the code freeze.) I think we want to have this DLL living in Wine for easier development, and probably dynamically load our custom Chromium fork from here. (just like MSHTML and wine-Gecko) The code for that fork could then be created in its own repo. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7032

2 2

[PATCH 0/1] MR7030: kernelbase: Don't try to print the path in the FIXME in GetTempPath2.
by Louis Lenders (＠xe) 16 Dec '24

16 Dec '24

Currently it gives weird output in the console logs like "GetTempPath2W (260, L"\2898\99f6\6fff") semi-stub", because the buffer is not yet filled with the path. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7030

3 2

[PATCH 0/2] MR7029: Fix regression in variable expansion.
by eric pouech (＠epo) 16 Dec '24

16 Dec '24

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

2 2

[PATCH 0/1] MR7026: win32u: Always update the surface region when the surface changes.
by Rémi Bernon (＠rbernon) 16 Dec '24

16 Dec '24

Instead of reapplying the old shape, which not only includes the window shape but also the old clipping and visible areas that can change independently. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57551 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7026

3 2

[PATCH v2 0/2] MR7013: odbc32: Add null pointer checks.
by Owen Rudge (＠orudge) 16 Dec '24

16 Dec '24

"erwin Data Modeler" crashes under Wine when making ODBC connections (to native ODBC drivers on macOS via iODBC) due to passing in `NULL` values in places. These checks allow execution to proceed successfully. -- v2: odbc32: Add null pointer checks to update_result_lengths helpers. odbc32: Avoid crashing if str is null in debugstr_sqlstr. https://gitlab.winehq.org/wine/wine/-/merge_requests/7013

3 3

[PATCH 0/1] MR7027: po: Update Japanese translation.
by Akihiro Sagawa (＠sgwaki) 16 Dec '24

16 Dec '24

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

2 1

[PATCH 0/2] MR7013: odbc32: Add null pointer checks.
by Owen Rudge (＠orudge) 16 Dec '24

16 Dec '24

"erwin Data Modeler" crashes under Wine when making ODBC connections (to native ODBC drivers on macOS via iODBC) due to passing in `NULL` values in places. These checks allow execution to proceed successfully. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7013

5 7

[PATCH 0/1] MR7025: winex11: Sync gl drawable outside of the win_data mutex.
by Rémi Bernon (＠rbernon) 16 Dec '24

16 Dec '24

Some thread might be destroying a window, calling destroy_client_window from release_gl_drawable, holding the GL context mutex and trying to enter the win_data mutex. At the same time, another thread might be moving its window, calling sync_gl_drawable from X11DRV_WindowPosChanged, holding the win_data mutex and trying to enter the GL context mutex. The deadlock was present before already, although less frequently triggered as sync_gl_drawable was done conditionally if the client window has been moved. This triggers now more frequently in the dxgi:dxgi tests. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7025

2 1

[PATCH v6 0/2] MR7023: Draft: ntdll: Fix SetCurrentDirectory behaviour on trailing periods
by Lois Gomez (＠loisgomez) 16 Dec '24

16 Dec '24

**Context:** Vanguard Saga of Heroes is an MMORPG that was closed years ago by SOE. However the game is still alive in the emulator scene at https://vgoemulator.net/ The game seems to have relied on an old behaviour of the function SetCurrentDirectoryW( LPCWSTR dir ) which currently crashes the game. This patch was derived from a solution discovered by the community at https://vgoemulator.net/phpBB3/viewtopic.php?t=5563 It was previously reported to wine bugzilla in this bug report. https://bugs.winehq.org/show_bug.cgi?id=34285 **Summary of changes:** Updates the function RtlSetCurrentDirectory_U to remove any trailing "." as the documented behaviour of the setCurrentDirectory function is to resolve the trailing period to the directory itself Previously the game will set the current directory as "bin\\." and subsequently crash when trying to access it as it is being returned as "bin\\.Caches". After this change It will correctly set the directory to "bin\Caches" and run successfully **Notes:** This is my first contribution to the project so I might be making a lot of incorrect assumptions on how this might affect other binaries. Please do let me know if there is a better way to implement a change like this in the codebase. Thanks for your help -- v6: add tests for RtlSetCurrentDirectory_U https://gitlab.winehq.org/wine/wine/-/merge_requests/7023

3 3

[PATCH v5 0/1] MR7023: Draft: ntdll: Fix SetCurrentDirectory behaviour on trailing periods
by Lois Gomez (＠loisgomez) 16 Dec '24

16 Dec '24

**Context:** Vanguard Saga of Heroes is an MMORPG that was closed years ago by SOE. However the game is still alive in the emulator scene at https://vgoemulator.net/ The game seems to have relied on an old behaviour of the function SetCurrentDirectoryW( LPCWSTR dir ) which currently crashes the game. This patch was derived from a solution discovered by the community at https://vgoemulator.net/phpBB3/viewtopic.php?t=5563 It was previously reported to wine bugzilla in this bug report. https://bugs.winehq.org/show_bug.cgi?id=34285 **Summary of changes:** Updates the function RtlSetCurrentDirectory_U to remove any trailing "." as the documented behaviour of the setCurrentDirectory function is to resolve the trailing period to the directory itself Previously the game will set the current directory as "bin\\." and subsequently crash when trying to access it as it is being returned as "bin\\.Caches". After this change It will correctly set the directory to "bin\Caches" and run successfully **Notes:** This is my first contribution to the project so I might be making a lot of incorrect assumptions on how this might affect other binaries. Please do let me know if there is a better way to implement a change like this in the codebase. Thanks for your help -- v5: kernelbase: Workaound for SetCurrentDirectoryW misuse https://gitlab.winehq.org/wine/wine/-/merge_requests/7023

3 2

[PATCH v3 0/1] MR7023: kernelbase: Workaround for SetCurrentDirectoryW misuse in Vanguard: Saga of Heroes
by Lois Gomez (＠loisgomez) 16 Dec '24

16 Dec '24

**Context:** Vanguard Saga of Heroes is an MMORPG that was closed years ago by SOE. However the game is still alive in the emulator scene at https://vgoemulator.net/ The game seems to have relied on an old behaviour of the function SetCurrentDirectoryW( LPCWSTR dir ) which currently crashes the game. This patch was a solution discovered by the community at https://vgoemulator.net/phpBB3/viewtopic.php?t=5563 It was previously reported to wine bugzilla in this bug report. https://bugs.winehq.org/show_bug.cgi?id=34285 **Summary of changes:** Updates the function SetCurrentDirectoryW( LPCWSTR dir ) to remove any trailing "." Previously the game will set the current directory as "bin\\." and subsequently crash when accessing "bin\\.Caches". It will now access "bin\Caches" and run successfully **Notes:** This is my first contribution to the project so I might be making a lot of incorrect assumptions on how this might affect other binaries. Please do let me know if there is a better way to implement a change like this in the codebase. Thanks for your help -- v3: kernelbase: Workaound for SetCurrentDirectoryW misuse https://gitlab.winehq.org/wine/wine/-/merge_requests/7023

3 2

[PATCH v4 0/1] MR7023: ntdll: Fix SetCurrentDirectory behaviour on trailing periods
by Lois Gomez (＠loisgomez) 16 Dec '24

16 Dec '24

**Context:** Vanguard Saga of Heroes is an MMORPG that was closed years ago by SOE. However the game is still alive in the emulator scene at https://vgoemulator.net/ The game seems to have relied on an old behaviour of the function SetCurrentDirectoryW( LPCWSTR dir ) which currently crashes the game. This patch was a solution discovered by the community at https://vgoemulator.net/phpBB3/viewtopic.php?t=5563 It was previously reported to wine bugzilla in this bug report. https://bugs.winehq.org/show_bug.cgi?id=34285 **Summary of changes:** Updates the function SetCurrentDirectoryW( LPCWSTR dir ) to remove any trailing "." Previously the game will set the current directory as "bin\\." and subsequently crash when accessing "bin\\.Caches". It will now access "bin\Caches" and run successfully **Notes:** This is my first contribution to the project so I might be making a lot of incorrect assumptions on how this might affect other binaries. Please do let me know if there is a better way to implement a change like this in the codebase. Thanks for your help -- v4: kernelbase: Workaound for SetCurrentDirectoryW misuse https://gitlab.winehq.org/wine/wine/-/merge_requests/7023

2 1

Re: [PATCH v2 0/7] MR6911: ntdll: Add WaitCompletionPacket.
by Zhiyi Zhang (＠zhiyi) 16 Dec '24

16 Dec '24

> NtAssociateWaitCompletionPacket() initiates an asynchronous wait on the object handle. When the wait is done (the object is signaled), it queues a completion to the associated I/O completion port. Make sense. I will revisit this MR after the code freeze. > Maybe, in the particular case of React Native, it only uses WaitCompletionPacket on manual reset events or other objects with no-op satisfied()? FWIW I couldn't find any reference to NtAssociateWaitCompletionPacket in https://github.com/facebook/react-native. I am testing a React Native sample application. If I remember correctly, the NtAssociateWaitCompletionPacket() is from some Microsoft DLL, coremessagingxp.dll. I didn't look into the type of object it waits on. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6911#note_90563

1 0

[PATCH v6 0/4] MR6904: ntdll.dll: Update NtQueryDirectoryFile to align with current Windows behaviour
by Eugene McArdle (＠eugenemcardle) 16 Dec '24

16 Dec '24

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. -- v6: ntdll: Simplify check that unexpected files were not found https://gitlab.winehq.org/wine/wine/-/merge_requests/6904

4 8

Re: [PATCH v2 0/2] MR6990: win32u, server: Fix some window surface shape sometimes not being updated.
by darkblaze69 (＠darkblaze69) 15 Dec '24

15 Dec '24

Unfortunately this (ce7b293c5b29111d256f5416bb44a6c77d6ef673) broke some dialogs: https://bugs.winehq.org/show_bug.cgi?id=57551 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6990#note_90561

1 0

[PATCH v2 0/1] MR7023: kernelbase: Workaround for SetCurrentDirectoryW misuse in Vanguard: Saga of Heroes
by Lois Gomez (＠loisgomez) 15 Dec '24

15 Dec '24

Context: Vanguard Saga of Heroes is an MMORPG that was closed years ago by SOE. However the game is still alive in the emulator scene at https://vgoemulator.net/ The game seems to have relied on an old behaviour of the function SetCurrentDirectoryW( LPCWSTR dir ) which currently crashes the game. This patch was a solution discovered by the community at https://vgoemulator.net/phpBB3/viewtopic.php?t=5563 It was previously reported to wine bugzilla in this bug report. https://bugs.winehq.org/show_bug.cgi?id=34285 Summary of changes: Updates the function SetCurrentDirectoryW( LPCWSTR dir ) to remove any trailing "." Previously the game will set the current directory as "bin\." and subsequently crash when accessing "bin\.Caches". It will now access "bin\Caches" and run successfully Notes: This is my first contribution to the project so I might be making a lot of incorrect assumptions on how this might affect other binaries. Please do let me know if there is a better way to implement a change like this in the codebase. Thanks for your help -- v2: kernelbase: Workaound for SetCurrentDirectoryW misuse https://gitlab.winehq.org/wine/wine/-/merge_requests/7023

4 4

[PATCH 0/2] MR7024: Disable native titlebar while lacking enough space for SM_CYCAPTION.
by Jactry Zeng (＠jactry) 15 Dec '24

15 Dec '24

On Windows, applications can disable titlebar by extending the client rect to a bigger rect which will cover the area of the titlebar from WM_NCCALCSIZE: https://learn.microsoft.com/en-us/windows/win32/dwm/customframe#removing-th… Our current x11 and Mac driver doesn't handle this case. Attached is a test case for reproducing the issue. [hide_caption.c](/uploads/46a725e32569a6a0a929a0f183cf9ed8/hide_caption.c) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7024

3 3

Re: [PATCH v2 0/1] MR7017: ntdll: Set the correct character map name for Shift JIS
by Akihiro Sagawa (＠sgwaki) 15 Dec '24

15 Dec '24

From my point of view, SJIS is correct. Here is an example of SJIS locale usage written by [Red Hat](https://access.redhat.com/solutions/727043) and [HP-UX](https://web.archive.org/web/20061206132454/http://docs.hp.com/en/UXL… (see Appendix A). Historically, until around 2000, the Japanese UNIX environment mainly used an encoding different from Shift-JIS, called EUC-JP (Extended Unix Code for Japanese). Therefore, with the exception of HP-UX, SJIS has rarely been used in locale settings. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7017#note_90546

1 0

[PATCH 0/1] MR7020: include: Add methods arguments annotations for DirectWrite types.
by Nikolay Sivov (＠nsivov) 14 Dec '24

14 Dec '24

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57476 Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7020

3 2

[PATCH 0/1] MR7019: gdiplus: GdipPathAddRectangle should close the path.
by Esme Povirk (＠madewokherd) 14 Dec '24

14 Dec '24

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

2 1

2025

2024

2023

2022

wine-gitlab