February 2025 - Wine-GitLab

[PATCH v22 0/2] MR7131: DIR command improvements: Make /o by itself behave equal to how it does in Windows
by Joe Souza (＠JoeS209) Feb. 5, 2025

Feb. 5, 2025

"DIR /o" (no order specifier after /o) hasn't functioned the same as it does on Windows. This change makes the functionality of /o by itself equivalent to behavior on Windows. -- v22: programs/cmd: Cleanup DIR /O logic and remove @todo_wine@ from working tests. https://gitlab.winehq.org/wine/wine/-/merge_requests/7131

4 5

[PATCH 0/1] MR7263: hhctrl: Fix window class name in HH_CreateHelpWindow.
by Alex Henrie (＠alexhenrie) Feb. 5, 2025

Feb. 5, 2025

Fixes: 3358d4d3244570637ed9db24c04faa43cee1f75e -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7263

3 2

[PATCH 0/1] MR7262: ieframe: Use wide char string literals.
by Alex Henrie (＠alexhenrie) Feb. 5, 2025

Feb. 5, 2025

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

2 1

[PATCH 0/6] MR7261: Splay tree part 3
by Zhiyi Zhang (＠zhiyi) Feb. 5, 2025

Feb. 5, 2025

For React Native. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7261

3 7

[PATCH v3 0/10] MR7254: windowscodecs: Move component info registry key cleanup to a common failure path.
by Nikolay Sivov (＠nsivov) Feb. 5, 2025

Feb. 5, 2025

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> -- v3: windowscodecs: Remove redundant guid-to-string conversion when writing readers registration entries. windowscodecs: Fix a typo in metadata readers registration helper. windowscodecs/metadata: Add a stub for WICIfdMetadataWriter. windowscodecs/metadata: Add a stub for WICExifMetadataWriter. windowscodecs/metadata: Add a stub for WICGpsMetadataWriter. windowscodecs/metadata: Add a stub for WICUnknownMetadataWriter. windowscodecs/metadata: Add registration information for the "Unknown" writer. windowscodecs: Add a stub for IWICMetadataWriterInfo. windowscodecs: Move an hkey handle to the component info base structure. windowscodecs: Move component info registry key cleanup to a common failure path. https://gitlab.winehq.org/wine/wine/-/merge_requests/7254

4 13

[PATCH v20 0/3] MR7131: DIR command improvements: Make /o by itself behave equal to how it does in Windows
by Joe Souza (＠JoeS209) Feb. 5, 2025

Feb. 5, 2025

"DIR /o" (no order specifier after /o) hasn't functioned the same as it does on Windows. This change makes the functionality of /o by itself equivalent to behavior on Windows. -- v20: programs/cmd: Remove @todo_wine@ from working tests. programs/cmd: Cleanup DIR /O logic. programs/cmd: Add tests patch from @epo. https://gitlab.winehq.org/wine/wine/-/merge_requests/7131

2 3

Re: [PATCH v3 0/3] MR7256: ntdll: monotonic_counter and NtQuerySystemTime performance improvements on macOS.
by Marc-Aurel Zent (＠mzent) Feb. 4, 2025

Feb. 4, 2025

On Tue Feb 4 17:55:14 2025 +0000, Tim Clem wrote: > I think this calls for a quick comment (or at least a note in the commit > message) as to why we're using this function. Added a comment, explaining the rationale there. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7256#note_93649

1 0

[PATCH v2 0/3] MR7256: ntdll: monotonic_counter and NtQuerySystemTime performance improvements on macOS.
by Marc-Aurel Zent (＠mzent) Feb. 4, 2025

Feb. 4, 2025

`mach_continuous_approximate_time()` has the necessary precision for win32 ticks and can be up to 4x faster than `mach_continuous_time()`. Also `clock_gettime( CLOCK_REALTIME, &ts )` calls always end up in `__commpage_gettimeofday( struct timeval *tp )`: ``` * frame #0: 0x00007ff806788763 libsystem_kernel.dylib`__commpage_gettimeofday frame #1: 0x00007ff8066709a3 libsystem_c.dylib`gettimeofday + 45 frame #2: 0x00007ff806678b31 libsystem_c.dylib`clock_gettime + 117 ``` These extra calls, setup and converting from one struct format to another costs another 60 CPU cycles and in my testing makes `NtQuerySystemTime` approximately 30% faster as well with this MR. This is a fairly hot code path (especially when using certain out-of-tree in process synchronization patch sets), so probably worth the optimization here. All of these APIs are available since 10.12. -- v2: ntdll: Replace '0' with 'NULL' in gettimeofday() calls. ntdll: Use __commpage_gettimeofday in NtQuerySystemTime on macOS. ntdll: Always use mach_continuous_approximate_time on macOS. https://gitlab.winehq.org/wine/wine/-/merge_requests/7256

2 4

[PATCH 0/2] MR7256: ntdll: monotonic_counter and NtQuerySystemTime performance improvements on macOS.
by Marc-Aurel Zent (＠mzent) Feb. 4, 2025

Feb. 4, 2025

`mach_continuous_approximate_time()` has the necessary precision for win32 ticks and can be up to 4x faster than `mach_continuous_time()`. Also `clock_gettime( CLOCK_REALTIME, &ts )` calls always end up in `__commpage_gettimeofday( struct timeval *tp )`: ``` * frame #0: 0x00007ff806788763 libsystem_kernel.dylib`__commpage_gettimeofday frame #1: 0x00007ff8066709a3 libsystem_c.dylib`gettimeofday + 45 frame #2: 0x00007ff806678b31 libsystem_c.dylib`clock_gettime + 117 ``` These extra calls, setup and converting from one struct format to another costs another 60 CPU cycles and in my testing makes `NtQuerySystemTime` approximately 30% faster as well with this MR. This is a fairly hot code path (especially when using certain out-of-tree in process synchronization patch sets), so probably worth the optimization here. All of these APIs are available since 10.12. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7256

5 12

Re: [PATCH v3 0/1] MR7115: Added support of case FileAllocationInformation in SetFileInformationByHandle
by Stéphane Bacri Feb. 4, 2025

Feb. 4, 2025

@julliard Thanks for taking the time to review. About the file size increase, it sounds like a direct consequence of what this function is supposed to do, since it shall allocate some space for a file. Can you detail your thoughts about it? About the tests, I guess you mean unit tests, right? Or did you mean tests on the parameters (say check that size is aligned with cluster size, or > to current size)? In the first case, I'm gonna try to add something, as digging a bit more the code showed that there is a basis. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7115#note_93641

1 0