May 2025 - Wine-gitlab - list.winehq.org

Re: [PATCH v2 0/1] MR8137: kernel32: Implement ReadConsoleInputEx
by Jayson Reis (＠jaysonsantos) 25 May '25

25 May '25

Hi there @epo thanks for the review! As I could not find any auto formatting tool configured, I tried maintaining the formatting around the code I was changing, do you mind point me out where did you find it different? I will change the calls to GetProcAddress, as there is no public header, this means that on console.c I cannot call the Wide function from ascii and just convert, right? Meaning I would need to implement the logic twice? About being racy, reading the docs I did not see anything special about it, it seems like a wrapper around the PeekInputConsole to be less blocking, something gives you the feel it should be reimplemented? Also, is it a good idea to copy the exe for tests and run on Windows to see how it goes? At least this showed something to me that my test and implementation seem to be wrong like when calling the function with nowait, and it is empty, the return should be false but GetLastError seems to be 0 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8137#note_104465

1 0

[PATCH 0/1] MR8140: pdh: Add stub for PdhEnumObjects{A,W}.
by Louis Lenders (＠xe) 25 May '25

25 May '25

Wine-bug: https://bugs.winehq.org/show_bug.cgi?id=58255 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8140

2 1

[PATCH 0/1] MR8139: user32: Add stub for SetProcessLaunchForegroundPolicy.
by Louis Lenders (＠xe) 25 May '25

25 May '25

Wine-bug: https://bugs.winehq.org/show_bug.cgi?id=58254 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8139

2 1

Re: [PATCH v2 0/1] MR8137: kernel32: Implement ReadConsoleInputEx
by eric pouech (＠epo) 25 May '25

25 May '25

a couple of comments: * please read and conform to [Wine Coding style](https://gitlab.winehq.org/wine/wine/-/wikis/Wine-Developer's-Guide/Coding-Practice#some-notes-about-style) * AFAICT ReadConsoleInputEx\[AW\] isn't defined in Windows SDK so header files shouldn't be changed * tests must reflect what Windows does, not what your implementation does (hence the generated failures in CI pipeline) * this should be implemented using proper (new or modified) IOCTL:s; there's no way this API can be implemented on top of kernel32's one without being racy (as your proposal is). That's the reason why MS introduced this new API. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8137#note_104458

1 0

Re: [PATCH v17 0/1] MR7706: powrprof.dll: Partially implement PowerGetActiveScheme for SteamVR - closed
by Robert Lippmann (＠rlippmann) 25 May '25

25 May '25

This merge request was closed by Robert Lippmann. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7706

1 0

[PATCH 0/1] MR8138: gdiplus: Unify region rasterization in GdipFillRegion.
by Esme Povirk (＠madewokherd) 24 May '25

24 May '25

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

2 1

[PATCH v6 0/2] MR8097: server: Remove redundant `frame` parameter from redraw_window().
by Jinoh Kang (＠iamahuman) 24 May '25

24 May '25

Related: !8095 and !8013. -- v6: server: Remove unused parameter `frame` from redraw_window(). server: Add `nested` parameter to redraw_window(). https://gitlab.winehq.org/wine/wine/-/merge_requests/8097

3 3

[PATCH 0/3] MR8136: Fix interaction between FreeConsole() and CRT std streams.
by eric pouech (＠epo) 24 May '25

24 May '25

The Aim of this MR is to fix https://bugs.winehq.org/show_bug.cgi?id=10941 In a nutshell: - CUI app starts, and CRT inits its std streams from console, - app calls FreeConsole() then AllocConsole() and expects the CRT I/O functions to be able to output to newly created console. It fails: - when run with wineconsole, as the inherited std consoles are closed in FreeConsole(). Added tests, but current implementation of kernelbase is correct. So, the fix is to force wineconsole to pass unbound console handles with standard inheritance (to they are not closed), - when run from unix console, the std handles were also closed. There's no unbound console in that case. The fix is not to close these handles in FreeConsole(). - the other cases from the bug entry work because cmd was created with unbound console handles. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8136

2 3

[PATCH 0/1] MR7803: kernel32: Add export for CreateFileMapping2.
by Etaash Mathamsetty (＠etaash.mathamsetty) 24 May '25

24 May '25

Rainbow Six Siege checks if the export exists but then doesn't actually use it -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7803

6 8

[PATCH v2 0/1] MR7920: winecoreaudio: revert unix_set_volumes
by Ada Ahmed (＠yretenai) 24 May '25

24 May '25

I was able to replicate the issue seen in [Bug 58113](https://bugs.winehq.org/show_bug.cgi?id=58113) on my M1. The issue stems from the usage of AudioDevicePropertyVolumeScalar, which the audio driver for the M1 does not support (at least so it appears.) Using AudioObjectIsPropertySettable allows for fast checking for this situation, including preemptively disabling main channel audio if it appears to be unsupported. -- v2: winecoreaudio: revert unix_set_volumes from f46e9b8f120605d984f4ae14b4c815a67fbd1b59 https://gitlab.winehq.org/wine/wine/-/merge_requests/7920

3 5